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

HTML, CSS-FLEX BOX

@Mediteran2910

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


Hello everyone, I think mobile design is not good, everyone having ideas how to fix that?

Community feedback

@MerMory-Alloc

Posted

Hi @Mediteran2910 great work here, also i like how you did the mobile design. however if you want to do it just like the design, all you have to do is to give padding to the second container and allow all the elements inside it to have the full width also remove any left and right margins for those elements. there s also another thing i noticed in your css file , that you did named all the html elements to reset some properties of theme here. for this there is better approch is to use * intead like this

* {
   margin: 0;
   padding: 0;
   border: 0;
   font-size: 100%;
   font: inherit;
   vertical-align: baseline;
}

Marked as helpful

1

@Mediteran2910

Posted

@MerMory-Alloc thank you very much for feedback, I appreciate it. So instead of copying css reset I can just add *{ with that values inside} and that is going to be fine?

0

@MerMory-Alloc

Posted

@Mediteran2910 well i didn't change much in the code. * just means "all the selectors" so it almost the same as your code.

Marked as helpful

1

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