Design comparison
SolutionDesign
Solution retrospective
any remarks are appreciated
Community feedback
- @denieldenPosted over 2 years ago
Hi Eslam, great work on this challenge! 😉
Here are a few tips for improve your code:
- add
header
tag and wrap thenav
for improve the Accessibility - add descriptive text in the
alt
attribute of the images and not simple flag word - to make it look as close to the design as possible increase the size of
h2
of countries cards - in the dark mode the text of search input is black and illegible
- if i click country card the details page not open
- instead of using
px
use relative units of measurement likerem
-> read here - if I type a query that doesn't give any results, nothing happens, try adding a "no results" message
- I would also add a query reset button, I find it very convenient
- in the filters there is no way to return to all countries after choosing a region, add an entry "all region"
- for the details of countries you can use
ul
instead multiplep
Overall you did well 😁 Hope this help!
Marked as helpful0@eslam-94Posted over 2 years ago@denielden thanks for your suggestions, i will fix them as soon as possible
1 - add
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