Melvin Aguilar 🧑🏻💻• 61,220
@MelvinAguilar
Posted
Hello there 👋. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
-
The use of
outline: none;in buttons should be reconsidered for the sake of accessibility. The outline is an essential visual indicator for users who navigate with keyboards or screen readers. Removing the outline can make it challenging for users with disabilities to identify which element is currently focused.Consider customizing it to better fit your design while preserving its visibility, for example:
outline: 2px solid blue;
- Prefer
min-height: 100vhoverheightto prevent component cutoff on smaller screens.
I hope you find it useful! 😄 Above all, the solution you submitted is great!
Happy coding!
1
