Design comparison
SolutionDesign
Community feedback
- @festsnusaPosted almost 2 years ago
Congratulations for completing this challenge!
Here are some feedbacks to how to improve your code:
- I see that you tried to use BEM names to classes. But img element is outside of main. You can add header, main and footer elements and give them according classes: .header, .main, .footer;
- .main__add class should be inside .main class, but there is no .main class;
- You can also add some transitions while hovering buttons.
I hope my feedback was worth your time. Happy coding.
Marked as helpful1@AlexEGPosted almost 2 years ago@festsnusa yup I thought that too, thanks for the feedback I really appreciate it
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