@nkhatri7
Posted
This is really well done! The javascript code is really efficient and just done well overall. The design is also really good, the only thing I'd call out is when the screen width is between around 700px to 1000px, the period filters are too spread out. I think it would be better if you did justify-content: center
and then spaced them out a bit with margin
(but still keep them rather centralised). Other than that, I can't really fault your code and design, keep up the good work!
Marked as helpful
@BenjaDotMin
Posted
@nkhatri7 Yep I had the same thought! Sadly those sizes are within tablet dimensions, so would be quite common to see it. But that could be a great way to fix it, thanks!
Thanks so much for the kind words, it helps!