Styled Markup Feedback!

[rijkvanzanten]

• [ ] An HTML document can only have one h1. Use a single h1 with nested span elements here instead
• [x] Two closing </span> for one open tag
• [x] There's a dedicated selector for the first letter of a sentence. Easier than another span with a class
• [x] Closing tags don't have attributes
• [x] Once you find yourself adding incremental classes (like P4 P6 etc), you're probably better off with the :nth-child selector
• [ ] Try to use more descriptive class names. H and OO don't really mean anything in context. I'd personally use an ID on the parent like #title-art and then style the individual letters with nth-childs, like #title-art > :nth-child(2) { ... }

• [ ] Noticing quite a lot of repeated code like font-family, size, weight etc. Similar to the previous point, you can simplify that by doing something like:

  #title-art {
  font-family: 'Montserrat', sans-serif;
  font-size: 90px;
  font-weight: bolder;
  }
  
  #title-art > :nth-child(1) {
  transform: rotate(); /* etc */
  }

Overall output looks pretty solid! Keeping the design simple helps you focus on getting the details right.