Chris• 240
@ChristopherParke
Posted
Hey there. Looks good! One main issue I see is that you have desktop html, and mobile html. That's not best practices. What you should do instead is have one html structure, and change it with CSS using media queries. I see you have media queries, so I would just think about how you can change the layout using them, rather than hiding and showing html elements. Cheers
Marked as helpful
0
Yokke• 640
@Jexinte
Posted
Hey @ChristopherParke,
Thanks for your feedback honestly I haven't thought about it like as mentionned it was my second try so I was just only focus on semantics and active states.
I'll keep your advice for my next challenge thanks again !
0
