@Iryna-Gry
Posted
Hi. You did a great job! Here a few points to improve:
- If you focus on HTML and CSS practice it's better to use semantic tags in markup, such as <main>, <section>, <h1> and other, and use less of <div> tags. Using semantic tags can help you with such framework like Next (which uses React) and semantics is really crutial there. Also it is easier to read the code if you use different structural tags and distinguish the roles between them.
- <h1> is needed here for accesibility reasons (more on accessibility [here](https://www.w3.org/WAI/fundamentals/accessibility-intro/))
- The component looks good, but there is no need of retyping the text (uppercase letters at the begining of each phrase needed) I hope, this feedback will help you in your development. Have a nice day, Iryna.
Marked as helpful