Submitted 12 months ago

GradientWave - A Stylish Landing Page

Moiz Azad• 90

@Moiz3012

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Results summary component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

1- Design and Layout: What are your thoughts on the gradient design and color scheme used in this landing page? Does the layout and responsive design work well on various screen sizes? Any suggestions for improvement?

2- Content and Clarity: Is the content clear and engaging for visitors? Any suggestions for enhancing it? Do you think the title "GradientWave - A Stylish Landing Page" effectively represents the project?

3- Coding and Implementation: Are there any specific parts of the code that you find difficult to understand or improve upon? Are there any best practices or coding conventions that you think could be applied differently or more effectively?

4- User Experience: How does the landing page feel in terms of user experience and interactivity? Do you have any suggestions for making the user experience even better?

5- Overall Impression: What are your overall impressions of the project? Is there anything else you'd like to share regarding the project's strengths or areas for improvement?

Community feedback

beowulf1958• 1,190

@beowulf1958

Posted 11 months ago

Congratulations on finishing this project! I have a few suggestions. First, I could not get it to work on my computer; the Summary side overflows the card, and I can't see the svg icons (they are hidden by the overflow on the other side.) I found making the ".same" just a bit longer revealed the icons, and making the container <div> a bit bigger fixed the overflow problem.

Next: the name of the project is very cool, but the style guide indicates ".card2" has a gradient background, and the button has an active state where it changes the background color to a gradient. Then it would be a GradientWave for sure.

Last but not least, best practices on coding and implementation recommend to keep the CSS "DRY" (don't repeat yourself.) Declare the font family in ".card" and you can eliminate it everywhere else. Let the cascade do its job.

Hope this helps! I look forward to seeing what you do next.

Marked as helpful

Moiz Azad• 90

@Moiz3012

Posted 11 months ago

@beowulf1958 thanks mate very informative for me.

Please log in to post a comment

Join 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