Order summary component done

Community feedback

• 

  cacosted•570

  @cacostedPosted over 2 years ago

  Hello Ricardo, your solutions looks great and the added feature with JavaScript is great.

  One thing that I noticed is that you used a border on the "change" link, this is not big deal but if you pay attention de text does a little jump when you hover in and out.

  There are different ways to approach this, but in this case I think the intended version was to simulate a default <a> style with the underline, you can do this with text-decoration: underline. Also another way to fix the jumping would be using outline instead of border.

  0
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Ricardo, congratulations for your new solution!

  Your solution seems great, something you can improve is the shadow, thats too evident and was the first thing I saw. The box-shadow is too much dark and strong, the secret to create best and smooth shadow you need to give it less opacity and more blur here’s a good value for this shadow box-shadow: 5px 6px 12px 5px rgb(0 0 0 / 5%);

  This is a good site to create good shadows and just drop the code into the css https://www.cssmatic.com/box-shadow

  👋 I hope this helps you and happy coding!

  0