P
Clément Creusat• 1,675
@ccreusat
Posted
Hello !
Nice job! Seems ok to me :)
Some improvements :
- You can replace your
<div class="title">
by a<h1>
, this will fix your acc. issue - Check the design if you can have a bigger line-height for the text.
- You could change how to handle margin on title and description as well: -- For instance, you can remove all margin on .title and .description -- Add a padding: 24px on your .text -- Add this to your code :
.title + .description {
margin-top: 24px;
}
Marked as helpful
0
Paul THEUREAU• 120
@Paul21777
Posted
@ccreusat Hello !
I have done everything that you suggested. Have to fix things so that it exactly correspond to the design
Thanks for the feedback !
1
P
Clément Creusat• 1,675
@ccreusat
Posted
@Paul21777 Looks awesome man ! :)
0