Design comparison
SolutionDesign
Solution retrospective
Any feedback are welcome
Community feedback
- @omerome83Posted over 2 years ago
Nice job. I just noticed two things to take a look at:
In your code where you have:
<h3>30-day, hassle-free money back guarantee</h3>, I would use the <h2> tag instead and then the next heading would probably be <h3>. Just so the headers are in order.Also, I didn't notice that your <Sign Up> button does anything noticeable when hovered. You can slightly change the color or add an effect to the button using something like the :hover selector.
Marked as helpful0@RedouaneAjgagalPosted over 2 years ago@omerome83 Thanks for your feedback i appreciate it, and i already have a very light hover on "Sign Up" button, maybe i need to make it more noticeable next time
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord