@danielmrz-dev
Posted
Hello @originalboss!
Your solution looks great!
I have a suggestion for improvement:
- For semantic reasons, only use one
<h1>
per page - this should represent the main heading/subject for the whole page.
The <h1>
to <h6>
tags are used to define HTML headings. <h1>
defines the most important heading. <h6>
defines the least important heading. And do not skip heading levels - start with <h1>
, then use <h2>
, and so on.
-
For the same reasons, use
p
for paragraphs instead ofspan
. -
And maybe check BEM methodology for naming classes. I have to say, I know you mean
button
, but thatdiv.butt
had me laughing 🤣
I hope it helps!
Other than those details, you did a great job!