Roman Filenko• 3,335
@rfilenko
Posted
Hey, nice job overall, a few notes to improve:
- class namings could be better, sometimes repeating css styles;
- be consistent with values (like margin: 150px 0 4em; padding: 1em 0 3.5em;), follow vertical rhytm(choose one, for example, margin-bottom for spacing between elements);
- add transition to links, buttons;
- check into css variables.
Keep practicing, Roman
1
Dulce• 115
@iamdulce
Posted
Thank you so much, Roman! I'll follow your suggestions and improve the project! :D
0
