Rahul.K• 445
@rahulkhimsuriya
Submitted
I would love to get your feedback.
Yinon Hever
@yinonhever
All comments
- Yinon Hever• 835
@yinonhever
Posted
Nice solution overall. My only note is that responsiveness needs to be improved: on mobile view padding needs to be added, and the filter area needs a little redesign so it won't look too dense on the small screen (see the mobile version of the design files).
0 - Arihant721• 100
@Arihant721
Submitted
How would we need to enable php processing on Github? I have one linked, in the event that the email is valid, to display a 'login successful' message.
Yinon Hever• 835@yinonhever
Posted
You don't need PHP to display login success message, you can do it with the frontend languages. The simplest way to do it is using the JavaScript alert() method, but if you want to customize the appearance of the success message, you can create such element with HTML/CSS and use JavaScript to show/hide the message.
0 - Michelle• 415
@michelleracho
Submitted
Any feedback is always appreciated 😀
Yinon Hever• 835@yinonhever
Posted
Nice solution. I love how you dynamically adjusted the max-height animation
0 - Elhoucin Olmostaf• 20
@Elhoucin20
Submitted
Yinon Hever• 835@yinonhever
Posted
Hey. I think the Vercel link is broken
0 - Nina• 290
@nina1012
Submitted
Yinon Hever• 835@yinonhever
Posted
Hey. The slider isn't working for me on desktop
0
