@Bayoumi-dev
Posted
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 helpful
@JoseTomas-GP95
Posted
Hi @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