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

Responsive result summary component using CSS Grid and CSS Flexbox

@Muhammad-Faizan-Soomro

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


Hey Everyone! This is my solution for Result Summary Component challenge by Frontendmentor.io.

I am little bit unsure of my styling skills and I ended up creating lots of classes. I know this issue can be resolved by using Tailwind CSS and I am learning it but for now I want some valuable advice on how can I improve my code and create less classes.

Please review it and give some valuable advice/feedback as feedbacks are highly appreciated and I am open to suggestions on the best way I can solve this challenge. Looking forward to hear the views of other coders.

Community feedback

@iamdylanmj

Posted

You have done a great job, I am also a newbie like your self, here is what I noticed from your code, you have tried to use svg images directly, it is alright but using image tag is better, because it is less complex unless you don't wanna change svg colors. The other things is, considering media queries min-width is suit for larger screens , assume you set min-width value as 700px, then the css rules only apply when the screen size equal to 700px or greater than 700px., and considering max-width, if you set max-width as 375px , the rules are gonna apply when the screen size is equal to 375px or less. So if you need to add specific screen size between two values like you tried to do in your media query the minimum value should be max-width:375px and maximum value should be min-width:1439, this will cause to apply css rules when the screen size between 375px and 1439px. I did my best to explain. overall you did a good job, please see how other people have completed the same challenge, you will have lot to learn. good luck.

2

@Muhammad-Faizan-Soomro

Posted

@iamdylanmj Thankyou so much it means a lot! I got your point. I will surely check out other solutions and try to learn a thing or two. Appreciated. :)

0

@mohammed-sarhad-ahmed

Posted

.reaction { margin-left: 30px; } .memory { margin-left: 30px; }

.verbal {
    margin-left: 30px;
}

.visual {
    margin-left: 30px;
}

button {
    margin-left: 30px;
    padding: 8px;
}

Just create one class instead . .Margin-30{ margin-left:30} and add them to the HTML

1

@Muhammad-Faizan-Soomro

Posted

@mohammed-sarhad-ahmed got it! Thankyou so much! :)

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