@pikapikamart
Posted
Hey, great work on this one. Layout in desktop looks great , the vector is just smaller than the design, the site is responsive and the mobile state looks great as well.
You already got great feedbacks which is really nice to see, just going to add some suggestions as well:
- Avoid using
height: 100vh
on a large container like thebody
tag as this limits the element's height based on the remaining screen's height. Instead usemin-height: 100vh
, this takes full height but lets the element expand if needed. - You don't really need to create a separate element for the mobile-vector, you could have used a
picture
element so that you could define inside there the images that you want to show based on a certain screen-size. - When using
alt
attribute, avoid using words that relates to "graphic" such as "logo" and others. Animg
is already an image/graphic so no need to describe it as one. But since all images here are purely decorative, hide them. Decorative image must be hidden at all times by usingalt=""
and extraaria-hidden="true"
attribute on theimg
tag. - The accordion is not accessible. Remember, interactive components uses interactive elements. By using
p
you are making it not-accessible. - You could use
button
in here, this will wrap the accordion question as well as the dropdown-icon ( to avoid extra markup, use the dropdown-icon as background-image for the button ). Thebutton
will have a default attribute ofaria-expanded="false"
and will be set totrue
via js, if the user toggles it and vice-versa. - If you don't like to use
button
, usedetails
element. This is already accessible and they are intended for dropdowns. - Your accordion-answer is just being hidden visually but it is still being picked up by other assistive tech. When hiding elements, either use
visibility
ordisplay
, include either one in the transition so that your element will be properly hidden or not.
Aside from those, great work again on this one.
Marked as helpful
@Al-Baraa-Bakri
Posted
@pikamart very helpful feedback :) .. Thanks <3