Martin Eliáš• 890
@martinelias1312
Posted
Hello @Okain0,
here is my suggestion.
- it´s better to use only one <h1>, beneficial for screenreader users.
Anyways good solution, keep it up !
Marked as helpful
1
Design comparison
Solution retrospective
I will be glad to any feedback, it will be interesting to listen to any point of view.
Community feedback
Mojtaba Mosavi• 3,760
@MojtabaMosavi
Posted
1- Everything is displayed overly big on the mobile view because you've set alot of fix values such a fontsize.
2- The car images on the learn more pages overflow horisontelly on the mobile view this because you've put the max-width on the container of the image not the image itself.
3- Spend some time reading about mobile first workflow since it helps a great deal with ahieving responsiveness much easier.
4- Nice job being creative with you solution and adding the extra pages.
Keep coding :=)
Marked as helpful
0
Ahemad Shaikh• 80
@ahemadShaikh
Posted
everything is good but when we hover on button then it blows more bcoz of box-shadow, and maybe if you give transition timing better for smooth effect that will be more efficient
Marked as helpful
0
Please log in to post a comment
Log in with GitHub
Join 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