@Tux3er-Isma
Posted
Hi Renan
Tux3er has reviewed your code 🐧
Congrats 🎉, it looks nice, but I will give you some tips for your next project 💡:
HTML 🦴:
-
In my opinion is not a good practice to add onclick in each element. You can do it with JS with the addEventListener()
-
Also you can put tabindex instead to use it after in css with :focus
CSS 🎨:
-
As I said before you can use :focus in the numbers. Is like click in JS
-
You can also add a transition to all the hover states
JS 🟡:
-
The function mark you can get it with only HTML and CSS (as I said before)
-
You have to put a return button to return to the homepage
That's all, have a nice coding ⌨️!!!!
Marked as helpful
@Renan-do-vale
Posted
@Tux3er-Isma Thanks for the advices. Initially, my goal was to use the addEventListener(), but I couldn't make it work, so I had to come up with a workaround to make it work.
