Skip to content
Submitted about 1 year ago

Testimonials Grid Section

Ahmad Hassan•310
@Ahmadhassan0
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Testimonials grid section
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Solution retrospective

You can look at my code and If you have any type of feedback on how can I improve my code quality I'll be very happy to look at your feedback😊

Community feedback

  • Melvin Aguilar 🧑🏻‍💻•61,140
    @MelvinAguilarPosted about 1 year ago

    Hello there 👋. Good job on completing the challenge !

    I have some suggestions about your code that might interest you.

    HTML 🏷️:

    • Wrap the page's whole main content in the <main> tag.
    • You might want to consider centering all components directly within the <body> tag or the element with the .testimonial-section class. Currently, on my desktop computer, it's not centered vertically.

    • The alt attribute should not contain the words "image", "photo", or "picture", because the image tag already conveys that information.

      If you want to learn more about the alt attribute, you can read this article. 📘.

    I hope you find it useful! 😄 Above all, the solution you submitted is great!

    Happy coding!

    Marked as helpful
    2
  • Ahmad Hassan•310
    @Ahmadhassan0Posted about 1 year ago

    @MelvinAguilar Thanks for pointing out my mistakes😊

    I'll use the <main> tag and will read the alt article blog for a better understanding

    0

Please log in to post a comment

Log in with GitHub

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