Skip to content
Submitted about 3 years ago

Using SCSS for landing page

Muhammad Fadll•320
@fadelun
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

i think this is enough,so please give your feedback for my code

Community feedback

  • Tesla_Ambassador•3,000
    @tesla-ambassadorPosted about 3 years ago

    Hello! I like the way you nailed the organization and the responsiveness is really on point! The only things I can note are:

    1. I think you forgot to add the background color... I usually do too sometimes 😂
    2. Umm.. With the mobile design, I think you should reduce the font-size of the texts in the articles so that they can fit well.
    3. For the HTML issue, you should just add an "alt" attribute just to say the name of the image for guys that use screen readers.
    Marked as helpful
    1
    Muhammad Fadll•320
    @fadelunPosted about 3 years ago

    @tesla-ambassador yeah I'm too tired to adjust the font-size and there is something I was missed,thanks for the feedback :D

    0
    Tesla_Ambassador•3,000
    @tesla-ambassadorPosted about 3 years ago

    @fadelun You're very welcome!

    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