Francisldn• 250
@francisldn
Posted
Hi @JakubStranianek, nice work on completing this challenge. I have just submitted my solution today for this challenge. I have a few comments below:
- Really love the "play" icon on hover. I will try to implement the same as well.
- Scrollbar for the "Trending movies" doesn't look aethestically pleasing, I chose to hide it for my project
- Bookmark functionality doesn't seem to work, ie. a user can't add or remove bookmarks
- I see that you have implemented "login" and "sign-up" components but they don't seem to be available on the hosted site.
- Really great responsive layout
- Search functionality works fine, good job.
Marked as helpful
0
JakubStranianek• 300
@JakubStranianek
Posted
@francisldn Hi thanks for your reply.
I didn´t implement bookmark funcionality so it doesn´t work.
Login and Sign up components are working but i want to have a look design comparision on frontendmentors page. So i disable these two components.
Thank you so much for your feedback
0
