Grid sectoin

Please log in to post a comment
Log in with GitHubCommunity feedback
- @hashmi7917
hi Omaima, i have some suggestions for you here it is. firstly dont give grid column n rows in fixed pixels , alternatively you can give "fr" ex: repeat(4, 1fr). by using pixels u cant get responsive design.
- Account deleted
Hey there! 👋 Here are some suggestions to help improve your code:
- To center you content to your page, add the following to your Body Element:
body { min-height: 100vh; display: grid; place-content: center; }
-
To better specify the main content of you site you will want to encase your entire component inside a Main Element.
-
The **names of each individual are headings so they should be wrapped in a Heading Element.
-
To enhance the semantics of your component, you want to wrap each individual testimonial component in a Figure Element, the individuals information should be wrapped in a Figcaption Element and lastly, the testimonial itself should be wrapped in a Blockquote Element.
Code:
<figure> <figcaption></figcaption> <blockquote></blockquote> </figure>
More Info:
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Join our Discord community
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