@Prithvi12890
Posted
Hello Chaitanya!
Amazing work with the advice generator! It is fast and really good. Your code is well organized and easy to read & understand too. The styling looks good, close to the given design.
Though the accessibility issues can be ignored, you can remove them with the following steps:
-
In card.js, you can try giving the button a title attribute, as you are using an image as a button and don't want any text, so..
<button className="btn" onClick={() => setClick(!Click)}> <img className="dice" src={Dice} alt="" /> </button>
to<button className="btn" onClick={() => setClick(!Click)} title="dice"> <img className="dice" src={Dice} alt="dice" /> </button>
-
In index.html try changing
<div id="root"></div>
to<div id="root" role="main"></div>
Hope this helps you and am looking forward to more works from you ✌
Marked as helpful