Skip to content
Submitted about 4 years ago

Mobile first using Flexbox & Grid, Sass + vanilla JS

P
Axseinga•540
@axseinga
A solution to the Interactive pricing component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hi there,

feedback appreciated!

Thank you.

Code

Please log in to post a comment

Log in

Community feedback

  • tediko•6,700
    @tediko

    Hello, Agnieszka! 👋

    I didn't expect you to finish this fast 😅 Your solution looks very good and also responds well. We've talk about it yesterday but I just want to link you some resources to help you understand BEM better. link1 - en - very straight forward, simple example. link2 - pl - this was my first video I watched about BEM. I think it is very descriptive but I think it is a bit long-winded.

    Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility. Your .wrapper can be a <section> and your .accordion should be a footer element.

    Personally I'd use parapgraph for .heading--secondary text. It is description and doesn't need to be heading. Your h1 heading is enough.

    Thanks for the shoutout in github readme! 😊 Good luck with that, have fun coding! 💪

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