Joseph• 770
@Jos02378
Posted
Hey @R0b3rtG, good job on this solution!
Some suggestions for you:
- Try to split your CSS styles into a different file so it improves the readability and maintainability of your code.
- You can try to give your
main
tag amax-width
to keep the structure and shape of the card. - You can try to follow a CSS naming convention called BEM. You can watch this video for more information Watch the video.
- You can try to use relative units like em in the future for padding. Here is a link to an article that explains when to use which unit see the article.
I hope this helps, good luck!
0
Robert Iancu• 0
@R0b3rtG
Posted
@Jos02378 Thank you for your feedback. I'll follow it as much as I can :)
0