order summary componenet with html & css

Community feedback

• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  Congrats on the solution☀️ I have also done this one and enjoyed it a lot. Here are a couple of things you can improve which I hope will be helpful! 👍

  First of all, don't use a tag for "Proceed to Payment" or "Cancel Order". They are not links, it's buttons. Buttons are useful when you want to create some interactivity on the same page, for example, a pop-up modal or form submission. Definitely, this is not a big project and this button has no purpose right now however and it could potentially be a button as well but less likely, it’s good to have that habit of differentiating buttons and <a> tags 😊 As you see yourself, you left href attribute empty because you didn't need it right? You are not redirecting anywhere.

  Also, make sure to wrap the entire code in the main tag instead of using the div, the one where the class is “form”. It will help to remove report issues and improve accessibility as well. If you are using, for example, header or footer tags, you can place them outside the main tag.

  The image with the alt "icon-music" should be used more as a decorative image, rather than being attached to any content.

  Alt attribute for the image is important in order to specify alternative text for the image in case it will not be displayed. Using alt attribute is good for not only accessibility but also SEO and for situations when the image is loading too slowly. 
If the image is just for decoration you can still write an alt attribute but leave it empty, such images don’t need any alt tag but you will need to also add aria-hidden=“true”. What aria-hidden does is that it removes the entire element from the accessibility tree.

  If otherwise, you need to use an alt tag to describe the image. To write an alt tag you need to describe the content and purpose of the image and try not to use words like “picture of” or “image of”.

  0