Product Preview Card mobile and desktop

Community feedback

• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 11 months ago

  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 of span.

  • And maybe check BEM methodology for naming classes. I have to say, I know you mean button, but that div.butt had me laughing 🤣

  I hope it helps!

  Other than those details, you did a great job!

  0