Order Summary Card using Hover Sates for Links

Community feedback

• 

  Roberto Yukio Miyamoto Arita•320

  @RyukioMiyamotoPosted over 2 years ago

  Hi Dan, great job. I looked into the why of display: block after my last comment and it seems the reason is that block elements take up all width available, while inline elements take only as much as they need to, which might make some whitespace visible.

  For the border-radius, my suggestion would be to give it to the container rather than individually (altough in some situations this solution may be better), and while that initially won't do the trick, adding overflown: hidden will solve it.

  Hope you find that helpful, have a great week!

  Marked as helpful

  1

  ---

  

  Roberto Yukio Miyamoto Arita•320

  @RyukioMiyamotoPosted over 2 years ago

  @RyukioMiyamoto as for the alignment of the text, almost all HTML elements have a default margin value, so giving your h2 a margin: 0 should solve it. The way I avoid this is to give all my projects a basic reset of

  *::before, *, *::after {
     margin:0;
     padding: 0;
     box-sizing: border-box;
  }
  

  Where * stands for all elements. As a matter of fact, there's a table for all the default browser values in case you didn't know (just like me, before this reply hahah). W3Schools | CSS default values

  Marked as helpful

  1

  

  Dan Bennett•60

  @inkfrombloodPosted over 2 years ago

  @RyukioMiyamoto Thanks once again for being Super helpful. I've seen the * code from other's projects, but had not yet read about what it meant.

  1