Hi there 👋, I’m Loai and this is my solution for this challenge. 🚀

Community feedback

• 

  Pipow•500

  @PipouwPieuwPosted over 1 year ago

  Hello Loai, well done once again !

  I noticed a small issue with your Proceed to payment button. Currently only its label is clickable but ideally we should be able to click anywhere on the button to trigger it. This is because you applied the styles to the wrapper element card__accept--button. Instead you should apply them to the <a> inside.

  Otherwise your component looks good! Well done :)

  Marked as helpful

  1

  ---

  

  Loai Esam•670

  @LoaiEsam37Posted over 1 year ago

  @PipouwPieuw Thank you for the Feedback

  0
• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted over 1 year ago

  Hello there 👋. Congratulations on successfully completing your fifth challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  HTML 🏷️:

  • The role="main" is unnecessary for element main

  • The main landmark role is used to indicate the primary content of a document. It can be added to an element by using role="main", but instead it’s preferable to just use the <main> element.

  • In that case, it’s unnecessary to make the main role explicit.

  Examples:

  <div role="main">
            *this is a valid way to define a main role*
  </div>
  
  <main>
           *but this is shorter and uses correct semantic HTML*
  </main>
  

  I hope you find it helpful ! 😄 Above all, the solution you submitted is great

  Happy coding!

  Marked as helpful

  1

  ---

  

  Loai Esam•670

  @LoaiEsam37Posted over 1 year ago

  @0xAbdulKhalid Thank you this is very helpful

  0