Design comparison
Solution retrospective
I am a newbie at Frontend development and this is my second project. I tried to take minimal help and put together a code that works. Although, I haven't been able to follow all best practices.
Ant feedback to help make the code more optimised and professional would be very helpful. Thank you.
Community feedback
- @rsrclabPosted almost 3 years ago
Hi, @cloudyay ~
Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.
Here are some of the tips I like to provide.
- I suggest you to try transition on hoverable elements like image, heading and creator name.
- Please try BEM for naming element classes. It will help you a lot on bigger projects.
https://www.frontendmentor.io/solutions/my-first-solution-on-chanllenge-V-4IzAivH
Here is my solution to this challenge, and if it can help you even a bit, it would be happy to me.
Cheers ~
Marked as helpful1@cloudyayPosted almost 3 years ago@tymren608 That is extremely helpful, thanks a lot for the valuable feedback. Appreciate it!
0@cloudyayPosted almost 3 years ago@tymren608 Also, I have added transitions for the mentioned elements, its not visible in the screenshot, can you have a look at the website and let me know what you think about it?
0@rsrclabPosted almost 3 years ago@cloudyay ~
The transition I mean is something like it was on my solution. On mouseover image, overlay appears a bit slowly, not blinking at once. Hope my words can help you a bit.
Cheers!
Marked as helpful0@cloudyayPosted almost 3 years ago@tymren608 Cool, understood, I'll implement that for sure
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord