Skip to content
Submitted over 4 years ago

Responsive Web Using Sass

Vijay Kumar•260
@VijayKumarKTG
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Fylo landing page with two column layout
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Solution retrospective

Anyone can help me out in refactoring sass code and suggestion on how to write more sass code more efficiently.

Community feedback

  • Ovidiu-Antonio•3,125
    @ovidiuantonioPosted over 4 years ago

    Hello Vijay,

    Very nice solution! The site is responsive and it looks very good! You did a great job! One tip to align the social icons from the footer inside that circle is to use flexbox, you make the li displayed flex and then use align-items and justify-content properties to align the icon, and also don't forget to add display: flex; to the ul too, to get the social icons in one row.

    As for the scss code, to write cleaner code and more efficient I think you should use the BEM methodology on the html, so every element will have a class. You can look at my projects if you need help with this, to see how I implemented that (it's not perfect but I do my job :) ).

    Happy coding! Keep going!

    2
    Vijay Kumar•260
    @VijayKumarKTGPosted over 4 years ago

    @ovidiuantonio Thanks for your feedback and your suggestions. They are really useful. Ok, I'll also like to look at your projects. Have a great time Buddy! 🤩😁

    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