@Prithvi12890
Posted
Hello Keshav!
If you want to remove the accessibility issue we can do it quickly as follows:
Since you do not have only one heading, use <h1>' tag instead of
<h2>i.e., change
<h2 class="firstpara center">Improve your front-end skills by building projects</h2>to
<h1 class="firstpara center">Improve your front-end skills by building projects</h1>'
And, coming to center your whole card, try using relative units like % instead of px for the top attribute in the CSS file. Like, you can try changing the top attribute in section ID from
#section { top: 100px; }
to...
#section { top: 50%; transform: translateY(-50%); }
Another alternative is to use a flexbox which will take away most of the work and make sizing and positioning easy and flexible. If you don't know flexbox, I recommend doing this short course on flexbox to get you started.
Everything else looks great!!. Hope these suggestions help you ✌
Good Styling!