Design comparison
Solution retrospective
Don't have many experiences using javascript yet. any feedback is welcome.
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi Rico, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
- To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is.
- every Html document must contain the main tag, so we can identify the main content, to fix this, wrap all the content with the main tag. HTML5 landmark elements are used to improve navigation experience on your site for users of assistive technology.
The rest is great!
I hope it helps... 👍
0@ShuliiiPosted about 2 years ago@AdrianoEscarabote Hi Adriano, thankyou for the feedback. For this kind of page, what should be my h1 element?
1@AdrianoEscarabotePosted about 2 years ago@Shuliii this is just a form, taking this into account we have to imagine that after the user fills in the form, the main content will appear! so basically in this form it is not necessary to use
h1
.but just wanted to remind you that the use of
h1
is very important!happy codding!
0
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