Raymart Pamplona• 16,140
@pikapikamart
Posted
Hey, awesome work on this one. Layout in general looks great.
Some suggestions would be:
- Avoid using
height: 100vh
on a large container especially thebody
tag, this makes the element's height limited to the current remaining screen's height. Instead, usemin-height: 100vh
, this takes full height and will expand if it needs to. - Also, I would not alter the
width
of thebody
tag, you could alter themain
. - The
alt
value for the vector image should have been left empty likealt=""
since it is only decorative, also add anaria-hidden="true"
attribute on theimg
element as well. - When an
img
acts as decoration only, use the above method, but if the image adds content to the site, then make a meaningfulalt
value. - Always include a
h1
on a webpage, for this one, you could useh1
on theorder summary
but really, it is not suited to beh1
therefore, you could make theh1
asr-only
text, you can look that one up on the net:> - The music icon also should have used the mentioned method above since it is decorative only. Also, when using
alt
attribute, avoid adding words that relates to "graphic" such as "icon, logo, image.." it is already an image so no need to describe it as one.
Aside from those, great work on this one again.
0