All comments

• 

  Egemen• 55

  @egemendemir

  Submitted over 2 years ago

  Feedback is appreciated. If there are any mistakes or bad practices in the code please let me know.

  

  Order Summary Page Using Flexbox

  2

  1

  0

  

  Patricia Hurst• 60

  @patricia-hurst

  Posted over 2 years ago

  Hello, just a few things to point out:

  • for the annual plan and price, either use a <span> to separate them or set the price into a separate tag entirely (such as another <p> and then use classes to style them separately). This allows for much easier styling
  • you have anchor tags that are styled and behave like buttons, so you should be using actual <button>'s with the anchor tags
  • look into using em or rem for padding (I would read up on the difference between them, part of it is user preference and partly sometimes one is better to use in a certain scenario over the other)

  Marked as helpful

  1