four-card-feature-section-master

Community feedback

• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted over 3 years ago

  Hi!, nice job, things to improve:

  1- Regarding you markup, you're using to h1 for it, it's totally ok but you can also achieve the same behavior with one, you can achieve through either a br tag or max-width on it.

  2. The card are not interactive therefore there is no need for the hover efftect.

  3. I recomand you spend some time studying naming conventions and start learning one because it makes your markup much more reusable and readable, my favorite is BEM.

  Keep coding :)

  1

  ---

  

  JorddI Suarez•130

  @jorsuapPosted over 3 years ago

  @MojtabaMosavi Thank you very much for your recommendation, I did not know the BEM methodology and it seemed very good to have a more ordered code, again thanks

  0