order-summary-component-main using css flexbox

Community feedback

• 

  P

  hcm•505

  @HcmwebsPosted over 1 year ago

  Hi, This is a good solution; however, I believe you can make it better by;

  1. To the body, I should remove the margins and set it to display grid or flexbox with flex-direction to the column.
  2. including landmarks. After the container, the next element should be NOT a div but a MAIN and t FOOTER for the attributions class. These should solve the landmark issues.

  I hope these pointers are helpful.

  Regard,

  Marked as helpful

  0