Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social proof section with Flexbox and SASS.

GGS 260

@Guilherme-Goncalves-de-Souza

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


🇧🇷 Deixe aqui seu feedback ou sua crítica construtiva!!!

en Leave your feedback or constructive criticism here!!!

Community feedback

Alamin 2,000

@CodeWithAlamin

Posted

Hi GGS👋 Great job on completing this challenge! 🥳

I would like to share a few suggestions on this solution if you don't mind.

  • Use semantic HTML tags: Some of the HTML tags used in the code, such as div, do not convey any meaning about their content. Using semantic tags, such as header, main, section, and footer, can help make the code more readable and improve the accessibility of the page.

  • Use BEM naming convention: The code uses class names that are not very descriptive, such as left, center, and right. Using the BEM (Block-Element-Modifier) naming convention can help make the class names more meaningful and easier to understand. For example, review__stars instead of stars and review__by-review instead of by-review.

  • Avoid using inline styles: Inline styles, which are applied directly to an element using the style attribute, can make the code harder to maintain and override. It's generally better to define styles in a separate stylesheet and apply them using class names or other selectors.

Overall, this is a very well done solution to the challenge. Great job!

I hope this feedback was helpful. 😊 Keep up the good work!👍

Marked as helpful

2

GGS 260

@Guilherme-Goncalves-de-Souza

Posted

Thank you very much!

Gratitude for all the feedbacks, I will take care of them in my next projects.

Thank you very much @CodePapa360 !!!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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