@correlucas
Posted
This comment was deleted
@Hamid210545
Posted
@correlucas Thanks ,..... i will put that in my mind and will practice it later... Thanks
This is an amazing project and it really test your css skills especially css most important property CSS Grid .... I like this project and I have done my best to achieve it ...... There is no difficulty in achieving the design ...... Frontend mentor is an amazing plateform
@correlucas
Posted
This comment was deleted
@Hamid210545
Posted
@correlucas Thanks ,..... i will put that in my mind and will practice it later... Thanks
@VCarames
Posted
Hey @Hamid210545, some suggestions to improve you code:
The profile images Alt Tags need to be improved. It should state the following; βHeadshot of -personβs full name-β
Remove the min-height
from .testimonials not need.
You want to add a third layout to make the transition from mobile π± -> desktop π₯ views smoother.
Using CSS Grid with Grid-Template-Areas will make things way easier when building the layout and give you full control of it.
Here is how it looks like fully implemented: EXAMPLE
Desktop View Code:
.testimonial-container {
display: grid;
grid-template-columns: repeat(4, 1fr);
grid-template-rows: repeat(2, 1fr);
grid-template-areas:
"daniel daniel jonathon kira"
"jeanette patrick patrick kira";
gap: 30px;
}
.daniel-card {
grid-area: daniel;
}
.jonathan-card {
grid-area: jonathon;
}
.jeanette-card {
grid-area: jeanette;
}
.patrick-card {
grid-area: patrick;
}
.kira-card {
grid-area: kira;
}
Happy Coding! π»π
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord