@PhoenixDev22
Posted
Hello Luciano Oliveira,
Your solution looks great. I have some suggestions regarding your solution, If you don't mind:
HTML
- About
<h1>
it is recommended not to have more than one h1 on the page . Multiple<h1>
tags make using screen readers more difficult, decreasing your site’s accessibility. You can add a<h1>
withclass="sr-only"
(Hidden visually, but present for assistive tech).
- In this challenge , the images are much likely to be decorative. For any decorative images, each img tag should have empty
alt=""
as you did andaria-hidden="true"
attributes to make all web assistive technologies such as screen reader ignore those images .
- What would happen when the user click those learn more?In my opinion, clicking those "learn more" would likely trigger navigation not do an action so button elements would not be right. So you should use the <a>.
- There are so many repeated style rules , better to use reusable and manageable classes. For example: each column have the same styles , So you can use a class .cardComponent for the shared styles, then for each distinct styles like (background color)use another class. . cardComponent 1, . cardComponent2 . cardComponent 3
- Really important to keep css specificity as low/flat as possible. It’s not recommended use the ids to target the DOM elements for styling purposes , using ID's creates problem due to the specificity , better to use class so that it could be more manageable and reusable. IDs have a much higher specificity than classes) IDs have many uses in a webpage aside from being a CSS selector. For example as page anchors, fragment identifiers or to link labels to form fields.
Aside these , Good work! Hopefully this feedback helps
Marked as helpful