Order Summary using fex

Community feedback

• 

  darryncodes•6,390

  @darryncodesPosted almost 3 years ago

  Hi Mohssine,

  Great little design there.

  I'd recommend removing background-position-y: -60%; and adding background-size: 100% auto; to better match the challenges design.

  All the best!

  Marked as helpful

  1

  ---

  

  mohssine96•90

  @mohssineoussamaPosted almost 3 years ago

  @darryncodes i will do and see , thanks anyway

  0

  

  mohssine96•90

  @mohssineoussamaPosted almost 3 years ago

  @darryncodes u have right it better now , i appreciate it

  0
• 

  Jonel Hatwell•585

  @hatwell-jonelPosted almost 3 years ago

  Hi! congratulations for finishing this challenge. there are some issue that saw checking your html code you must use anchor tag <a> to the webpage when you are link to a certain content so you must change the "proceed to payment" button to anchor tag. also there are issues in the accessibility you can check that on the report page.

  Article on correct using of anchor tag:

  https://medium.com/design-code-repository/a-vs-button-b859547cae4d

  keep coding 👍👍

  Marked as helpful

  1

  ---

  

  mohssine96•90

  @mohssineoussamaPosted almost 3 years ago

  @hatwell-jonel thanks for your comment i will work on my mistakes thanks again

  0