Results summary component

Community feedback

• 

  Paweł Chudecki•840

  @SoulRvr29Posted over 1 year ago

  You can center your .container in a better way. Add to body:

  body{
  min-height: 100vh;
  display: grid;
  place-content: center;
  }
  

  or if you want to use flexbox:

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

  Then you can remove in .container margin: 95px 400px and position: fixed. Also remove the properties in .attribute, you don't need them.

  Avoid setting elements to a fixed height. The width is better to be set with max-width. Good luck, and happy coding!

  Marked as helpful

  1

  ---

  

  Keo Lamola•60

  @KeoLamolaPosted over 1 year ago

  @SoulRvr29 I really appreciate your feedback and time you took to give me advice. Thanks, will make changes to the code.

  0