@mattstuddert
Posted
Hey Daniel, I'm glad you enjoyed this challenge! You've done a really good job getting everything working nicely. Here are a few pointers after taking a look at your code:
- For the region filter, you're currently hardcoding each item. Another approach would be to have an array of the regions
['Africa', 'Americas', 'Asia', 'Europe', 'Oceania']
and then iterate over it to generate the items. This would reduce your code quite a bit. - Also, I'd recommend breaking out your code into smaller components. This makes the code much more manageable and maintainable.
- For your CSS breakpoints, you're currently using
max-width
media queries. For a future project, I'd recommend having a go at usingmin-width
as it means mobile users load in fewer styles.
I hope these tips help. Let me know if you have any more questions! 🙂