Order Summary Component using CSS custom properties

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello, CodeyBrodey! 👋

  Good effort on this challenge! 👍

  Regarding your question, when I wasn't a PRO member, I would try guessing the value for the border-radius. Not only the border-radius but also the amount of padding and margin (and much more). So, I guess the only way is to keep trying until it looks similar to the design. 😅

  Also, don't worry too much about it. It's okay if the site is not looking the exact same as the original design. As long as it looks similar then it's good enough. I would recommend focusing more on the code quality instead. 😉

  Some feedback from me.

  • I would not recommend using viewport units for border-radius. It is because the viewport units are relative to the user's screen size. So, some users might see the card have more rounded corners and some might not be able to see the border-radius.
  • Use single class selectors for styling whenever possible instead of id. id has high specificity which can lead to a lot of issues, especially on the larger project. It's best to keep the CSS specificity as low and flat as possible. 😉

  I hope this helps! 😁

  Marked as helpful

  0

  ---

  

  CodeyBrodey•150

  @CodeyBrodeyPosted over 2 years ago

  @vanzasetia Thanks! Very helpful feedback.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @CodeyBrodey You're welcome! I am glad that you found it helpful! 😄

  0