Submitted almost 2 years ago
REST Countries API with color theme switcher
@mohamed-montaser1
Design comparison
SolutionDesign
Solution retrospective
this is country website, this website actully take 2 days to complate, this was amazing and hard a little bit, please if you have any edit about my website tell me in comments...
bye🙂...
Community feedback
- @BirukAbrehamPosted almost 2 years ago
Hello Mohamed,
You did an excellent job bravo! Here are some suggestions
- On the country flag cards you could make the entire card clickable ( At the moment only the country name is clickable)
- You can retain the state of the theme (dark/light) in the browser storage so that when the users refresh the page, they don't lose the theme they chose. ( I suggest you use Local Storage to keep the state of the theme)
- To get ride of "Page should contain a level-one heading", wrapper the React root div with main tag
1
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