Design comparison
SolutionDesign
Community feedback
- @grace-snowPosted 21 days ago
Good job on this!
Only a few pointers really
- there's no benefit to using a header here. It's not banner content so that element will be treated exactly like a div, a generic element. There's no benefit to using section to wrap the questions either. This whole design is only showing one block of content. If anything, all of this should be in a section, including the title.
- the list of FAQs would benefit from being in an actual list.
- all images in this are decorative so should have empty alt text.
- you don't actually need to swap out elements in the js. Just rotate those icons instead
Marked as helpful1
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