Lucas Cerri• 1,120
@llKryptonixll
Posted
Hey nice work, but I would also call the function after the page has loaded, so that a random advice can be seen after the page has loaded and not an empty area. window.addEventListener("load", getAdvice); Or just call the function without the load event.
The HTML looks very good but i would replace:
<div class="dice">
<img src="./images/icon-dice.svg" alt="">
</div>
with:
<button aria-label="Get random advice button">
<img src="images/icon-dice.svg" alt="Dice Icon">
</button>
This is better for screenreaders and accessibility
Marked as helpful
0
Fitri Amelia• 140
@fitriamelia150
Posted
@llKryptonixll hi, thank you for the advice it is helpful for me..
0