@AdrianoEscarabote
Posted
Hello Sebastian, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:
images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty alt=""
and add aria-hidden="true"
attributes to make all the assistive technologies of the Web, as screen reader. Learn the differences between decorative/meaningless images vs important content.
Since this project is only based on a single page component, there is no need for a h1 tag. It's always a good idea to prevent accessibility errors, so I believe it would be beneficial for you to add a "h1" in this component. Don't worry if you forget about "h1," though; it's a good practice for when you are developing larger sites.
<h1>Order Summary</h1>
avoid using px
, If your web content font sizes are set to absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. if you want to keep using px for development and then format the whole code to rem, you can use this vscode extension: px to rem
The remainder is excellent.
I hope it's useful. 👍