Responsive Result Summary Page

Community feedback

• 

  Rohit Deshpande•870

  @rohitd99Posted over 1 year ago

  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

  ---

  

  Saeed Ahmed•10

  @saeedahmedasadPosted over 1 year ago

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

  0