Hi @ShahzarMazhar !
I think your project looks good.
Just a couple of things from me.
I would keep working on variable names.
For example, here
dark-color and color-not-white are bad variable names.
Is it a dark grey, red, blue?
I really like tailwind css’s approach to naming colors.
You could do something like this
That way when you start working on larger projects, you can create a separate stylesheet for the different shades of colors you are working with and other developers will know what those variables mean.
Another thing to notes is I would avoid using inline styles.
<section class="main-section" id="About_SSH" style="margin-top: -5em;">
You should practice separation of concerns and keep your html and css code in separate files.
Also, remember that inline styles take precedence over external style sheets.
You could get into a frustrating debugging situation where you try to change some styles in your css but it is not working. Then you realize it is not taking effect because you have applied inline styles for that element and that is why your changes are not showing up.
Overall, it is just best to keep them seperate.
Hope that helps!