@AdrianoEscarabote
Posted
Hello Shankarganesh-B, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:
Since this project is only based on a single page component, there is no need for a h1 tag. It's always a good idea to prevent accessibility errors, so I believe it would be beneficial for you to add a "h1" in this component. Don't worry if you forget about "h1," though; it's a good practice for when you are developing larger sites.
<h1>HTML & CSS foundations</h1>
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
The rest is spot on.
Hope it’s helpful to you. 👍