Rio Cantre• 9,710
@RioCantre
Posted
Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…
- For best practice, wrap this line
<article class="nft-card">
which is the starting point of the content withmain
tag. Refer it with this one Semantics - Make a separate ruleset for
.nft-card__text
and addcolor: var(--soft-blue);
or view this line&__image-wrapper &__text
in the Sass file if,
is needed in between - Organized the HTML structure
Other than that...
- The project is responsive and utilized the details based on the original design
- Well done with the hover state of the design, specially the hero image
- Great job in using Sass for styling, the composition is organized and proper usage of sizing elements
- The alignment of details and elements are done accordingly
Above all, The project is awesome! Keep it up!
1
Esteban• 150
@ehmenzala
Posted
@RioCantre Thank you so much for the feedback! 🖤
0