Hi! Mihailo, I just saw your work and test your responsiveness. You did it really cool and even it is fully functional too.
But your design is not fully responsive. I think you need make it responsive for more user accessible.
I can see you have 3 accessibility issue and the reason is just add main tag inside the App component at top or you can simply change div with main tag. It will solve your accessibility issue.
Overall Really nice.
Good Luck
Marked as helpful
@TheDilp
Posted
@skyv26 Thanks for the tip, I've added the main tag and it fixed 2 of the issues, the last being the requirement of an h1 tag which is not part of the given design.
Regarding responsiveness, that is something I'd implement in the future if I were to continue with this project. The challenge was to replicate the designs at the given breakpoints that the style guide provided, which are 375px and 1440px so I stuck to those instructions. Though I completely agree with you, it would be something to work on further.
Cheers and thank you for the feedback