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

Responsive Layout using Css Flexbox

Praise 70

@Prayhaiz

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

raya 2,850

@rayaatta

Posted

Hi, congrats completing the challenge Have you noticed that your images Don't show up anywhere. Here is a solution. Replace

<img class="logo" src="/images/icon-suvs.svg"> ```
With 
```html
<img class="logo" src="images/icon-suvs.svg">

You don't have to add that "/" Infront of the image because the image folder is in the Same location as the current html document. Happy coding.

0

Praise 70

@Prayhaiz

Posted

Thanks I will do that now @rayaatta

0
raya 2,850

@rayaatta

Posted

The other thing your html needs to be a bit more semantic you should wrap the main content of the page in a

<main></main>

And the other inside a

<Footer></footer>

Using semantic html eases work for screen readers and improves search engine optimization Check out this article about semantic html https://developer.mozilla.org/en-US/docs/Glossary/Semantics @Prayhaiz

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