Design comparison
Solution retrospective
Hello everyone, I would like to know if something is right or something failed, any comment is very helpful, thank you!
Community feedback
- @ApplePieGiraffePosted almost 4 years ago
Hey, Augusto! 👋
Good work on this challenge! 👍
Like axevldk mentioned, it would be a good idea to add the background images to the pages and either include a tablet layout or wait a little while before switching to the mobile layout. 😉
Keep coding (and happy coding, too)! 😁
And happy holidays! 🎄
1@Augusto248Posted almost 4 years ago@ApplePieGiraffe
Hi ApplePieGiraffe, thanks for your reply.
Like you and axevldk mentioned, I will take into account about waiting a little while before switching to the mobile layout.
Happy holidays! 🎄
1 - @axevldkPosted almost 4 years ago
Hi, Augusto ~ I have studied your work and good work so far. Here are some of my opinions.
-
There is no background on the whole page.
-
Star rating section is different from design. You can give margin-left to individual sections, or can use grid in this case. Also border radius and background-color is missing.
-
And I think cards to go mobile mode too early. Hope it to maintain desktop vision until 769px or 993px.
Hope this will help you even a bit. Happy coding ~ ✨
1@Augusto248Posted almost 4 years ago@axevldk Hi axevldk, thanks for your reply. 😁
I will take into account what you have suggested.
Happy holidays! 🎄
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