Skip to content
Submitted over 1 year ago

Used Flexbox, tried to do some class based design.

Scynes•190
@Scynes
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Results summary component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Any tips/suggestions would be greatly appreciated! I'm not the best at CSS, but I'm trying to get a firmer grasp on cleaner code!

Community feedback

  • jorditosa•120
    @jorditosaPosted over 1 year ago

    Hi!

    Very good design, some feedback: Reverse to use flex at body to place at center the component, and grid for the main container.

    Like html and css file structures and writting also hover effect on summary results.

    Probably, in terms to be exactly as design proposal, some text colors and lower font sizes could be better. And for classes, using some methodology ( interesting one is BEM por example ) could be better to have those better readable and ordered.

    Marked as helpful
    1
    Scynes•190
    @ScynesPosted over 1 year ago

    @jorditosa Hey, thanks for the feedback

    0

Please log in to post a comment

Log in with GitHub

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