HTML, CSS-FLEX BOX

Community feedback

• 

  MerMory-Alloc•250

  @MerMory-AllocPosted over 1 year ago

  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•90

  @Mediteran2910Posted over 1 year ago

  @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•250

  @MerMory-AllocPosted over 1 year ago

  @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