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 design made with flex and grid.

@asamoahe3

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I will like to have feedback from CSS gurus on how to improve my skills. Any recommended sources to learn more on CSS will be appreciated.

Community feedback

@asamoahe3

Posted

Thanks @CyrusKabir. I'm not proficient in css. I'm still leveling up myself in web development. I really appreciate your concern for reviewing my codes too.

1
CyrusKabir 1,885

@CyrusKabir

Posted

hello my dear friend ♥ your card it's good and clean but here some problems :

  • don't use percentage % value in width and try use max-width in absolute units like em rem px and control it in differernt sizes with @media but at all don't use % in width you can see what happend in bigger size screens in developer tools. i mean useing percentages in child element it's ok and actually it's a power to use that but in container " % " it's not the answer .
  • and also why you use overflow scroll in default in your card ?? i mean have you ever any componnet in the web in mobile screen which have scroll in default i am not sure about that it's not in design too
  • have some comment in your css it's make it more readable and more easier to maintain
  • and at last try to avoid " !important " in css and go read about " css specifity " at all you did good but try to fix this issues to sharp your skill
1

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