anass-lamiri• 320
@Anass-Lamiri
Posted
Hi! First of all, you did a great job in your challenge, I only want to add some pointers to polish your solution:
- in the mobile viewport you need to make the image full width and without border-radius like in the design image for mobile, here is a solution that you need to add to the media query:
.container__img {
margin: 0;
width: 100%;
height: 12rem;
border-radius: 0;
}
- also for the mobile viewport the content should take the full screen :
.container__img {
margin: 0;
width: 100%;
height: 12rem;
border-radius: 0;
}
- the last thing is the preparation time section the span is the one that should have that color and the text should have the color provided in the design.
I hope you find this feedback helpful, and feel free to reach out.
Marked as helpful
1
Vinicius Duarte• 60
@ViniCellist
Posted
@Anass-Lamiri Thanks! I appreciate your feedback!
0
Vinicius Duarte• 60
@ViniCellist
Posted
@Anass-Lamiri I've just fixed the margin and border-radius! Thanks again!
0
anass-lamiri• 320
@Anass-Lamiri
Posted
@ViniCellist You're welcome.
0