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 score result for mobile width (375) and pc screen

@davidlayobo

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I appreciate feedbacks cause am still a learner

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Hi there! Your site looks pretty good and mobile responsiveness works as expected.

Some suggestions I have are:

  • using a <main> tag to wrap around the main content of your page
  • remember to import your font using Google Fonts. Right now your site is just using a default font. Google Font Link: Here
  • if you wanted to center your container I suggest you to use flexbox instead of position absolute. In your case you could do:
body{
display: flex;
flex-direction: column
justify-content: center;
align-items: center;
}

And then you could remove your position: absolute stuff from your container's css. Hope you find this feedback useful 👍

Marked as helpful

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