Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive Landing Page with Pure CSS

@ThutaZaw2603

Desktop design screenshot for the Manage landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


This is the first challenge that I do from frontend mentor. Unfortunately, my site would not be responsive on tablets from 800px to 430px viewports. Can pay feedback and tell me any problem you see on my site. Thank You !

Community feedback

@zineb-Bou

Posted

Hi there,

Using only @media screen and (max-width:428px) you are not covering all screen sizes, which makes the content get smashed together then scaling down to a small screen size

You shouldn't include more than h1, Apply the correct heading level

Avoid setting an explicit Height to element, this won't allow the container to shrink when content grows inside it.

When the image is just for decoration hide it from the screen readers using aria-hidden: true

I dropped some notes after I checked your code quickly.

Happy coding.

Marked as helpful

1

@ThutaZaw2603

Posted

@zineb-Bou Thanks you so much Sir!. I also found many issues after I have submitted. On the further projects, I will surely use more break points.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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