Design comparison
SolutionDesign
Solution retrospective
Tried my best to work on responsive design. Any suggestion on best practices or points where I can improve more or any sorts of feedbacks are welcome.
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, great job again on this new challenge. Site looks great in desktop, it is responsive as well and the mobile state looks great.
Again, some suggestions would be:
- You don't need to
display: none
the.attribution
since that is a meaningful content of the page, instead what you want to do is that, on thebody
tag just addflex-direction: column
and make the.attribute
display: block
again. - Decorative images on this challenge could have use an extra
aria-hidden="true"
attribute so that they will be totally hidden alongside with thealt=""
- On this challenge, you can use the
FAQ
as anh1
for now. But if you think it is not suited to be anh1
then make sure to add anh1
to the page as it is a requirement. Since there are no text-content that are visible that could beh1
, you will make theh1
screen-reader only text. Meaning this will be hidden for sighted users and only be visible for screen-reader users, search aboutsr-only
stylings and see how it is used. Theh1
text should describe what is the main content is all about, thish1
would be placed as the first text-content inside themain
element. - Now, you don't want to wrap the
FAQ
inside with the dropdown, it should be sitting on its own row:
<div class="body_container"> <h2>FAQ</h2> # own row The containers in here </div>
- Using
p
tag ordiv
to trigger the accordion is not accessible. Interactive components uses interactive elements. By usingdiv
you are making it not-accessible. - Instead on this, to avoid extra manipulation, you could use
details
tag. This is suited for accordions and it is accessible right away so you don't have to create extra things to make it accessible. - The dropdown
svg
should be using as well thearia-hidden="true"
since it is decorative only.
Aside from those, the site looks great.
Marked as helpful1 - You don't need to
- @NikeshGamalPosted about 3 years ago
Thank you @pikamart for your detailed feedback. Your feedbacks are so detailed and helpful. Helped me a lot adapting better approaches . Appreciated a lot.
1
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord