Skip to content
Submitted almost 2 years ago

responsive fun flex project

sass/scss
warrenbarney•70
@warrenbarney
A solution to the Social proof section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I had issues with the stars is this a good way to use svg's? Any advice would be great! Thanks.

Code

Please log in to post a comment

Log in

Community feedback

  • P
    Alexandra•580
    @Alexandra2888

    Hi! Congrats for finishing challenge! Your solution looks very close to the design. Things I think you can improve:

    -in HTML: you can use section for every wrap component. it's better to use more semantic tags, it's also improving readability than using a lot of divs. For div class="attributes" I suggest using footer tag. You can use svg in many ways, see a detailed article from FreeCodeCamp here: https://www.freecodecamp.org/news/use-svg-images-in-css-html/

    -in SCSS: you can use *::after and *::before to target also pseudo-elements. Also implementing BEM naming convention in your classes would allow you to nest scss classes and improve your code readability. Happy coding!

    Marked as helpful

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