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

sefat khan 120

@Sefat-Khan

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Rate my code. I'm new here and i'm learning web development. Plz let me know what should i have to improve.

Community feedback

P

@gladstone28

Posted

Your submission on the Results Summary Component Challenge is very good but I noticed the following which you could look at: CSS:

In your CSS, you've defined a font-family as 'HankenGrotesk', but you are not using this font-family anywhere in your styles. You can apply it to specific elements by using the font-family property.

In the CSS for your icons, you are using the .h3 class selector, which should be .icon h3 to target the h3 elements inside the .icon divs.

In your media query (@media (max-width:600px)), you've made some font and container size adjustments, which is good for responsive design. However, you can also consider making the button width 100% to ensure it fits within the container at smaller screen sizes:

Marked as helpful

0

sefat khan 120

@Sefat-Khan

Posted

@gladstone28 thank you for your advise

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