Franklin David Macias Avellan
@Davic15All comments
- @RbkHerediaSubmitted over 1 year ago@Davic15Posted over 1 year ago
Hello Rebeca.
In this project you also have to modify the <div id = "root"> </ div> by <main id = "root"> </ab>, remember for semantics reasons.
Each image must have a text (attribute) with the description of the image, because if the image does not work, the text is shown, in addition the Screen Readers use this to interact with the user who have some type of visual disability.
You have an error with Card Tag, there is no card, you must create it as a component or you must change it for another container.
I hope I've helped.
0 - @RbkHerediaSubmitted over 1 year ago@Davic15Posted over 1 year ago
Hi. Good job. Looks great.
I see that you have some problems with your accessibility report, to correct them you must do the following, firstly in your index.html file in the <div id="root"></div> tag, change it to <main id="root "></main>.
In addition, in the images you use, you must add the alt attribute inside the src tag, it allows you to write the description of the image, in case the image does not work.
For reasons of semantics with the labels, you have problems in your report.
Modify that, let's see how it goes.
Marked as helpful0