@pikapikamart
Posted
Hey, awesome work on this one. Layout looks good in general.
Marlon already suggested some great feedbacks, just going to add some in here as well:
- Always have a
main
element that will wrap the whole main content of your webpage.main
element helps users to properly navigate your website. On this one, thesection
could be replaced withmain
. - The
alt
for the vector image of the card should have usedalt=""
on it. If an image only acts as a decoration, usealt=""
on it, but if the image adds content, then use a meaningfulalt
value. - If you use a heading tag, make sure that the lower level heading tag is present. In your case, you use
h4
, but there are noh2, h3
present before it, this creates an issue, so when using heading tag, make sure to use it incrementally by one, start fromh1
toh2
toh3
. - The other 3 interactive elements on this one could have used
a
tag since this component is not really aform
wherebutton
are used. - The
proceed to payment
, there is no need to usep
tag inside it, just use thep
tags text as the text of the element itself.
Aside from that, really great job on this.
Marked as helpful