@ponyma2100Submitted over 2 years ago
Filip Skoczeń
@untlsnAll comments
- @untlsnPosted over 2 years ago
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 helpful0