Skip to content
Submitted almost 4 years ago

Responsive Landing page using CSS Grid

Kazer Panol•170
@kazerpanol
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

Hi. This is my first task here in frontendmentor. May i request for your feedback on how can i improved my codes. Thanks.

Community feedback

  • Katie•405
    @kem522Posted almost 4 years ago

    Hey there, nice work!

    Two suggestions for you:

    1. The alt text on the images should be more descriptive, both for accessibility and SEO purposes. You can read more about that here.

    2. Similarly, for the blocks of text you should use <p> tags instead of <div>. Good semantic HTML is beneficial for accessibility and SEO! You can read more here.

    Keep up the good work!

    1
    Kazer Panol•170
    @kazerpanolPosted almost 4 years ago

    @kem522 Hi. Really Appreciate your feedback.☺️ I will apply those suggestions on my future projects. Also thanks for the reference.

    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