Skip to content
Submitted almost 3 years ago

Meet Landing Page With Grid and Flexbox

Anublissā€¢460
@Anubliss-0
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Meet landing page
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I would like to ask the community if there was a cleaner way to implement the four photos in the main section of the page?

One thing I wanted to note, the paragraph text had low contrast making it an accessibility issue. I adjusted the text to be darker in order to make the site contrast friendly. To do this I used the WAVE accessibility tool https://wave.webaim.org/

Any feedback appreciated!

Community feedback

  • Travolgi šŸ•ā€¢31,400
    @denieldenPosted almost 3 years ago

    Hi Anublis,

    a cleaner and easier way to implement the four photos is to use flexbox and style the img tag directly

    Happy coding :)

    Marked as helpful
    1
    Anublissā€¢460
    @Anubliss-0Posted almost 3 years ago

    @denielden Thanks for the feedback! I'll try that out and see how easy it is for me to implement.

    1
    Travolgi šŸ•ā€¢31,400
    @denieldenPosted almost 3 years ago

    @Anubliss-0 Let me know how it goes ;)

    1

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