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

@Pepeleeno

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


making it mobile friendly was such a pain in the ass 🤯

Community feedback

@samaelwebdev

Posted

your project look awesome, I took a quick look at your code and have two suggestions:

instead of:

html, body, div, h1, h2, h3, h4, h5, h6, p, a, main { margin: 0; padding: 0; border: 0; }

use:

  • (asterisk) { margin: 0; padding: 0; border: 0; }

more on CSS selectors (https://www.w3schools.com/cssref/css_selectors.php)

instead of using pixels try using rem, it will help with responsiveness... if you are not familiar with rem you can use a px to rem converter online...

happy coding

Marked as helpful

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