@AdrianoEscarabote
Posted
Hey Kishore Kongani, how’s it going? I was really impressed with your project’s result, though I have some advice that could be helpful:
You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader.
Consider using rem
for font sizes. When font sizes are set in absolute units like pixels, users can't adjust the text size based on their preferences. Relative units like rem
adapt to the screen size and user settings, making them more flexible across various devices.
If you'd rather keep using px
, you can download a handy VS Code extension that converts pixels to rem
automatically
link -> px to rem
Everything else looks great.
Hope this helps! 👍