Results Summary Component

Community feedback

• 

  Bryan Li•3,530

  @Zy8712Posted about 1 year ago

  Your site looks pretty good. I think a better way you could've organized the scores on the right side of the page is using flexbox (instead of the right and left properties you used). Other things I think I should mention is:

  • you should included a <main> tag to wrap around your page contents for accessbility purposes
  • the <h1> tag should be used for mainly the most important text on the page (e.g. the title of the page, which in this case is "Your result"). The headers have default font sizes which can be changed in css using the font-size property
  • you should also use flex box to center your card (instead of the top: 150px you have right now), it could look something like this:

  body{
   width: 100vw;
   height: 100vh;
   display: flex;
   flex-direction: column;
   justify-content: center;
   align-items: center;
  }
  

  Besides that I'd say it looks pretty good for your first challenge. Nice work 👍

  Marked as helpful

  1

  ---

  

  Nicole•20

  @niko-niko-nomPosted about 1 year ago

  @Zy8712

  Thank you for the feedback, I genuinely learned a lot from it! :)

  0