@Zy8712
Posted
Your looks great! Good job!
The only thing I'd change is the hover effect on the button so that it looks closer to the original design (unless you changed it on purpose). The original has: the button's background turn transparent on hover, the button text turns white, and the button has a white border.
One other thing worth mentioning is that you should only really have one <h1>
tag per page in your website. So my suggestion would be to change the headesr to <h2>
tags and then have a hidden <h1>
tag that can be used as a title for the page.
Hope this helps š
Marked as helpful
@itzvikashgupta
Posted
sure! thanks will work on it asap! @Zy8712