@dliferova
Posted
Hello, great job! I would advise you
- to use semantic HTML markup (such as section, main, etc.), it will help you maintain better organization and improve code readability.
- be mindful of having only one h1 heading per page in your markup.
- sometimes you style elements based on classes, and sometimes based on ids; it's better to stick to one approach and style elements only by classes.
- additionally, avoid using abstract names like id="a" or class="karma"; try to denote the element's semantics through the code, for example, class="karma-card", which immediately indicates that it's a card. You can use multiple classes on one element and eliminate ids, for example, class="card-s karma-card".
- I also noticed that the "supervisor" card has a narrower width than the others; it might be something worth correcting.
Good luck with your future projects!