@GnandalSubmitted 2 months ago
Angelo Silva
@angelo-sigoAll comments
- @angelo-sigoPosted 2 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 helpful0 - @deborapatraoSubmitted 3 months ago@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:
- Try reduce the use of
div
tag by using semantic tags likemain
andsection
. - 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 helpful0 - Try reduce the use of