Social proof section solution. Mo'men Kamal
Design comparison
Solution retrospective
Hey 😃, I used BEM methodology as @Tymur Renhach told me, Maybe it's not a big project but I felt that my code is more organized now. I didn't know what to name the class for that element that contains '.title' and '.content' div's, At first, I named it '.title-and-content-container'🤯 but then I named it '.footing' 😅,
so in a situation like this what would you do? and at which degree should I be specific at naming CSS classes? did I use BEM correctly? and overall what do you think about my solution ❤?
Community feedback
- @FoxMalder-coderPosted almost 3 years ago
Hi! I think "lead" or "intro" will be good names. Your BEM is not bad, but some things could be improved. For example, review-card__info and review-card__info-2 is not really informative. I think it's better call them review-card__author, review-card__author-name, review-card__author-photo and so one... And for review-card__content I suggest review-card__text.
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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