Gnandal Elisée• 170
@Gnandal
Submitted
Angelo Silva
@angelo-sigo
All comments
- Angelo Silva• 60
@angelo-sigo
Posted
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
@deborapatrao
Submitted
Angelo Silva• 60@angelo-sigo
Posted
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
divtag by using semantic tags likemainandsection. - 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 - Try reduce the use of
