Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Pure css and html done with atom text writer.

@TSmithThe2nd

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I have some phantom space in my center div that I can't get to go away. Any advice or resources to look at to help me figure it out would be appreciated.

Community feedback

Ken 935

@kenreibman

Posted

To get rid of accessibility issues as well, I would wrap the entire content starting from div class="header" in a main tag like you did for your footer

For linking images, I would keep the files in the image folder that frontendmentor provides you. There's no need to extract it all in one place like you did in your github.

In your Index.html I would then link images like so: <img src="images/icon-supervisor.svg">

0
Ken 935

@kenreibman

Posted

I believe having margin: 2rem on .card is one of the factors of your "phantom space"

I also applied:

.middle {
    ...
    ...
    display: flex;
    flex-direction: column;
    align-items: center;
}

which seems to perfectly space out your cards.

I also recommend setting max-width with set values such as rem to keep your card sizing more consistent, and scaling it down as your screen gets smaller/bigger.

Using % for the width on .card and .middle shrinks the width too much when scaling down.

I hope this helps!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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