Order summary component with CSS Flex

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted almost 3 years ago

  An anchor element cannot be nested within a button or vice-versa. On the second point, it looks like it is an anchor element instead of a button

  Marked as helpful

  1
• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted almost 3 years ago

  Nice work Darren.

  For the buttons and the anchor:

  • <a href="#">Change</a> , what do you think , selecting this change link/button does? It appears as if the button has to open maybe a form so that you can make some changes. If that is correct then that has to be a button
  • <button class="pay">Proceed to Payment</button>, on this one , this maybe be a navigation to the Payment page .

  Well if you have some suggestions on the buttons let me know

  Happy coding

  Marked as helpful

  1

  ---

  

  Darren Wong•110

  @wongd-hubPosted almost 3 years ago

  Thank you, @ChamuMutezva!

  Great point on the <a> tag best being a <button>, I'll have to pay closer attention to what each component should be doing in the future.

  As to your second point; did you mean that the button should perhaps have a hrefattribute since its purpose is to link to something?

  Thank you again!

  0