Result-summary-components

Community feedback

• 

  Rajaul Islam Ratul•1,250

  @ratul0407Posted about 1 year ago

  @BigNet90 congratulations on completing this challenge, You've done a great job🎉🎉

  But I think you might consider some changes :

  • You don't need to give the container a specific width like, you have set 100em remove it from the container and by doing this you are depending on the content of your container to set the width. So the width will be set according to the box-1 and box-2. And it looks so much similar to the design
  • But you need to center the container on your page and to do that just set

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

  and it will center the entire result summary at the center of the page

  • you are getting some side scrolling issues to solve that in your body tag just set
    overflow : hidden
  • And the box shadow is too dark try some light colors on the box shadow instead of gray and even maybe change the values a little bit and you are all set to go

  I hope it seemed helpful to you🙂🙂

  Happy coding my friend👍🎉🎉

  Marked as helpful

  0

  ---

  

  Abubakar Sadiq•100

  @BigNet90Posted about 1 year ago

  @ratul0407 thanks I really appreciate I hope we can link up so I can learn more from you

  1

  

  Rajaul Islam Ratul•1,250

  @ratul0407Posted about 1 year ago

  @BigNet90 sure🙂

  0