Filip Skoczeń• 370
@untlsn
Posted
Nice job. I recommend adding some debounce to the button, because the user may click many times and the advice doesn't change and it may be considered a bug You can make the button gray for some time, or replace the dice with the waiting time
You can work with semantics You can change advice-container from div to e.g. article, title to header, attribution in footer to footer and wrap Home with main
Marked as helpful
0
