All comments

• 

  Iva•120

  @IvuskaSubmitted over 2 years ago

  Hi everyone,

  I have fixed names of CSS classes.

  Thanks a lot for any feedback :)

  

  Social proof section (SASS, Flexbox, mobile-first solution) v3

  2

  50
  

  Iva•120

  @IvuskaPosted over 2 years ago

  Hi @grace-snow,

  thank you a lot for your feedback, helped me a lot and I learned new things:) I fixed the issues and split the component to smaller components to prevent the nesting problems.

  Regarding the names of the classes I followed up this article https://blog.frankmtaylor.com/2021/10/21/a-small-guide-for-naming-stuff-in-front-end-code/ shared to me in Frontendmentor newsletter vol. 71. It looked a little bit strange also to me but I gave it a chance and was not aware that it could possible problem for potential employers... Maybe I will have to go deeper to this naming convention area :-)

  0

  This comment has received 1 replyShow