@RioCantre
Posted
Hello! Good job on implementing the hover state effectively. The design is pretty close to the original but I could suggest the following for you:
- Inside the
.creator-text
rule set add amargin-top: 1rem
or equivalent to align it on the center - Its better to have
margin-left: 0
andmargin-right: 1.5rem
in the.creator-img
to balance the space and make it align to the rest of the content. - The icons should align to the text, in order to do that you could try
.price {
color: var(--primary-2);
font-weight: 400;
display: flex;
justify-content: center;
}
.time-left {
font-weight: 400;
grid-area: D;
justify-self: end;
display: flex;
justify-content: center;
}
img {
margin-right: 0.5rem;
}
-
The
hr
tag should have a width of 300px to give full length on the inner content. (only applicable if you made changes on the above suggestions.) -
The images should have
alt
attribute to give emphasis whenever it fails to upload. -
Wrap the
.attribution
infooter
tag to make it more readable. -
Clean the whitespaces in your code.
-
Use a validator to check any errors in your solution.
Keep up the good work ! I hope this helps.
Marked as helpful
@Yuko-code
Posted
@RioCantre
Thank you so much for the detailed advices !! This will help me a lot :)
@RioCantre
Posted
@Yuko-code You can mark it helpful if it really does ;)