Design comparison
SolutionDesign
Community feedback
- @Bayoumi-devPosted over 2 years ago
Hey Tomas,
My suggestions:
- Using more than one
<h1>
is allowed by the HTML specification, but is not considered a best practice. Using only one<h1>
is beneficial for screenreader users. --- Multiple<h1>
elements on one page - Add
cursor: pointer;
to thebutton
, The cursor indicates to users there is an action that will be executed when clicking on it.
I hope this is helpful to you... Keep coding👍
Marked as helpful0@JoseTomas-GP95Posted over 2 years agoHi @Bayoumi-dev.
Thank you for leaving your comment.
I was able to see your code to know what was the best way to solve it. I saw that you have an h1 but you decided to hide it. You hide it by this point:
Use only one <h1> per page or view. It should concisely describe the overall purpose of the content.
is that why you hid it? Do you think it is necessary to hide an element that you will not use?
I thank you for giving me new knowledge and I ask you to continue evaluating my next projects. I appreciate it
0 - Using more than one
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