Design comparison
Community feedback
- @skyleranglhPosted about 1 month ago
Good job on completing the project. Here are some recommendations that could be implemented.
The main content is not centred responsively. The sections have extra padding/margin around them so they're not lined up with the main heading. There is a little bit too much spacing on the accordion text. You can't navigate anything on the page with tab other than the links at the bottom. Manually setting the inline style for the main element has caused unintended overflowing of the page; too much height. I recommend using relative units (em, rem, %, etc) to style the page as it is not quite responsive. The code needs to be formatted with consistent spacing, and unused variables/console logs need to be removed.
Marked as helpful0
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