@danielmrz-dev
Posted
Hello @ekahanny!
Your solution looks great!
I have a couple of suggestions for improvement:
- For semantic reasons, and since that is the main title of the screen, you can replace the
<h3>
with<h1>
. This change is not just about the size of the title.
The <h1>
to <h6>
tags are used to define HTML headings. <h1>
defines the most important heading. <h6>
defines the least important heading. Only use one <h1>
per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>
, then use <h2>
, and so on.
- Also, you can add a
hover effect
to the title, to indicate that's a clickable element. Just a color shift and cursor indicator is enough:
.title:hover {
color: yellow;
cursor: pointer;
}
I hope it helps!
Other than those details, your solution is great!
Marked as helpful
@ekahanny
Posted
@danielmrz-dev thank you for your feedback