All comments

• 

  s4nd0•320

  @s4nd0Submitted almost 2 years ago

  I'd love to hear your comments on my solution.

  

  Order Summary using CSS flexbox

  1

  20
  

  Mahmoud Elgendy•220

  @mahmoudelgendy1Posted almost 2 years ago

  Hello developer ☺️,

  You did it in a great way and I have some comments that might be useful to you :

  1- Did you know the following facts ?

  • Over 80% of internet users use mobile devices to surf the web.
  • 83% of mobile users expect a flawless experience whenever they visit a website with any mobile device.
  • Up to 70 percent of web traffic comes from mobile devices.

  This is why we all should use the Mobile First Approach : A “mobile-first” approach involves designing a desktop site starting with the mobile version, which is then adapted to larger screens (contrary to the traditional approach of starting with a desktop site and then adapting it to smaller screens).

  2- It would have been better if you did this:

  • font-weight: bold; for .proceed
  • background-color: transparent for .content-annual
  • more left & right padding for .order-content

  Great job!, I hope you find this helpful & please share your opinion if you disagree with me, I am still learning too.

  • Thanks for your time 👋

  Marked as helpful

  1