order-summary-component

Community feedback

• 

  Rakshith J•240

  @rakshithjodukallu

  Posted almost 3 years ago

  Hey, Go through Your work and its amazing. checked your design and solution its 100% matching. Only one thing recognised as for mobile view its not responsive, hope you will fix this. keep posting your design like this mate.

  01

  Marked as helpful
• Account deletedPosted almost 3 years ago

  Hey @Slideur, great job on this project!

  Some suggestions to improve you code:

  For accessibility purposes, its better to use rem/em instead of px for your CSS property values.

  To help keep your CSS code organized and easier to use, I suggest implementing CSS Variables. This will come in handy when building large websites, using light/dark mode, etc…

  It’ll look something like this:

  :root {
    --primary-color: value;
    --secondary-color: value;
    --tertiary-color: value;
  }
  

  And to use the variables you’ll use the var() function. So it’ll look like this.

  h1 {
  color: var(—primary-color);
  }
  

  Heres are some articles regarding CSS Variables.

  You can also take a look at my projects and see how I use it.

  https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_custom_properties

  https://www.w3schools.com/css/css3_variables.asp

  Happy Coding!

  01

  Marked as helpful