Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Results-summary-component-No-Responsive

FlowerPot30β€’ 100

@FlowerPot30

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Abdul Khaliq πŸš€β€’ 72,660

@0xabdulkhaliq

Posted

Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

  • I have a suggestion regarding your code that I believe will be of great interest to you.

CSS 🎨:

  • Looks like the component button hover state background has not been set yet.
  • Just add the following css rules
button:hover {
    background-image: linear-gradient(#6542FE, #342CE2);
}
  • Now your component's button background for hover state has been set properly

.

I hope you find this helpful πŸ˜„ Above all, the solution you submitted is great !

Happy coding!

0
Akib Razaβ€’ 900

@akibraza91

Posted

Hello there!

Congrats on completing the challenge! βœ…

Your solution is really impressive!

I've got a couple of ideas (about how to use HTML better) that could make it even stronger:

Think about using <main> to wrap your main content instead of <div>.

Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.

Hope that's helpful!

Keep up the great work!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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