Skip to content
Submitted over 3 years ago

Html, Sass, Grid, FlexBox, JS

Giancarlo Guerra Farfan•160
@GiancarloGF
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Pricing component with toggle
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Hello world! In this challenge I have learned more about pseudo-elements and properties (like :checked and :hover) that helped me to manipulate the other element's styles without using JS.

I hope to receive from you feedback or recommendations to improve my code in this challenge.

Take care!

Community feedback

  • Kent Hefley•10
    @KentHefleyPosted over 3 years ago

    You have a solid understanding of SASS. Very nice job. The only critique I can offer for the design: I would have added a box shadow to the card elements. The border on your .main class is a little too dark. Maybe go with a dark gray instead of black. I might suggest using more semantic tags like "main" and "article" in your HTML. I am guilty of creating div salads as well. Overall, nice work.

    0
    Giancarlo Guerra Farfan•160
    @GiancarloGFPosted over 3 years ago

    Hi and thanks for the feedback, and from now I'll use more semantics tags, you're right! Take care bro!

    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