@AdrianoEscarabote
Posted
Hey Shakthivel Ramesh Nirmala, how’s it going? I was really impressed with your project’s result, though I have some advice that could be helpful:
Using only <div> elements in code reduces the semantic meaning of the HTML structure, making it harder for assistive technologies like screen readers to interpret the content. This impacts accessibility, as specific tags (like <article>, <header>, <section>) provide clear context about the purpose of content sections. Additionally, it can negatively affect SEO and make the code harder to maintain or understand for other developers. Therefore, using semantic tags enhances both user experience and code quality.
Consider using rem
for font size .If your web content font sizes are set in absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. Relative units “stretch” according to the screen size and/or user’s preferred font size, and work on a large range of devices.
if you want to continue coding with px
, you can download a very useful extension in vscode, it converts px
to rem
!
link -> px to rem
Everything else looks great.
Hope this helps! 👍
Marked as helpful
@shakthivel-rn
Posted
@AdrianoEscarabote Thanks a lot for the detailed review. I should definitely improve myself on the accessibility side!