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 Result Summary Page

@saeedahmedasad

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@rohitd99

Posted

Hi Saeed Ahmed

Congrats on completing the challenge 🥳.

I see that you've used your headings in an incorrect way. Generally headings are used from h1 through h6 and each page must only have a single h1 for the title. So I would suggest you the h1 for the "Your Result", and h2 for the "Great" and "Summary". Secondly you've used the height: 100vh on your body and the main in media query, I would suggest using min-height : 100vh so that the minimum height is 100vh and it can expand further as per content.

Hope it helps.

1

@saeedahmedasad

Posted

@rohitd99 Thanks for your feedback. Will try to overcome these mistakes in future

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