@Code-Beaker
Posted
Hi there, congratulations on completing this project... You've done a great work! 🎉
I have taken a look at your live site and your source code and I would like to forward some of my suggestions regarding your solution. ⬇️
- The "learning" element is not a heading. So, you can either go with an
a
tag or simply ap
and style it to match the design. - Please be aware that the heading tags should be hierarchical meaning that they should decrease by one level only. Here's a basic structure:
h1
h2
h3
...
...
- For the profile component, I recommend not using a
section
tag and instead something like adiv
- Use
class
to style the components. I have noticed that you're using the tag/element selector in your CSS to style the components. Instead, give aclass
to all of them and select them using the class. Also, please avoid usingid
for styling. Here's an article that covers why IDs aren't meant for styling. - Use
rem
for properties likeborder-radius
andbox-shadow
as well.rem
units are both easy to handle and are better when building responsive sites.
I hope you find these helpful... Happy coding! 😄