P
Clément Creusat• 1,675
@ccreusat
Posted
Hello and Nice job!
Hope it will help :
- For the button, for accessibility purpose, I would add aria-label="text" on the button. "text" could be something like "generate random advice"
- For the position of the footer, maybe remove the
position:absolute
and add agap:4rem
(more or less) on the<body>
. I think for the demo of this challenge, would be enough :)
Enjoy coding!
Marked as helpful
0
turtlecrab• 550
@turtlecrab
Posted
@ccreusat Thank you for your feedback! I didn't know about aria- properties, will look deeper into them!
0