All comments

• 

  Gnandal Elisée•190

  @GnandalSubmitted 3 months ago

  

  Blog Preview Card

  1

  00
  

  Angelo Silva•60

  @angelo-sigoPosted 3 months ago

  Hey there! How is you are going? I saw your solution and it is pretty good! But you forgot something important, in the design files has a style for hovering the card tittle. Another thing, it's a good idea/practice to give structured class names, you can make it by following some CSS architecture like BEM. Again, your project is very good, but always can be improved. If you like my feedback, pls give me a one too! See you soon!

  Marked as helpful

  0
• 

  deborapatrao•10

  @deborapatraoSubmitted 3 months ago

  

  qr-code

  1

  40
  

  Angelo Silva•60

  @angelo-sigoPosted 3 months ago

  Hi Debora, are you good? Your solution is very nice, but I saw somethings that can be improved in your code. I'll list them bellow:

  1. Try reduce the use of div tag by using semantic tags like main and section.
  2. It's a good practice to give direct and clear CSS class names.

  And that's it! Again, your solution is very good but always can be improved. If you like my feedback, please see my solution and give me one too! See you next time!

  Marked as helpful

  0