Skip to content
Submitted over 2 years ago

SunnySide Agency

Josh Dā€¢160
@Josh-D18
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Agency landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

What are some things that I could do better code wise? How can I get my design to match the mockup more closely.

Community feedback

  • Miguel Silvaā€¢510
    @migsilva89Posted over 2 years ago

    Hi @Josh, I will advise you to use max-width on your body. To avoid that the squares get messy for bigger screens.

    Keep up the good work!

    regards Miguel

    Marked as helpful
    2
    Josh Dā€¢160
    @Josh-D18Posted over 2 years ago

    @migsilva89 Thank you so much! I will make sure to do this!

    0
  • Shashree Samuelā€¢9,260
    @shashreesamuelPosted over 2 years ago

    Hey good job completing this challenge

    Keep up the good work

    Your solution looks great however I think that your card titles needs to be a bit bigger and have a heavier font weight

    In terms of your accessibility issues simply wrap all your content between main tags

    I hope this helps

    Cheers Happy coding šŸ‘

    Marked as helpful
    1
    Josh Dā€¢160
    @Josh-D18Posted over 2 years ago

    @TheCoderGuru Thank you so much!! I will implement those changes!

    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