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

Using Flexboxes for responsiveness(updated alts and sections)

Drew Hosick• 60

@drewhosick

Desktop design screenshot for the Clipboard landing page coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Still feel I'm missing a few things to make this a perfect project.

Community feedback

Vanza Setia• 27,855

@vanzasetia

Posted

👋Hi Drew Hosick!

I have some feedback on this solution:

  • For the logo image, you don't need to put the "logo" word for the alternative text. It's already inside the img tag. Just use the company name for the alt value.
  • The download buttons should be a link, since it will download something.
  • For any decorative images, you should leave the alt="" empty and add aria-hidden="true" to make the screen readers ignore those images.
  • About your commit message that says "final changes", I would not recommend to do that, instead you should give your commit message with more meaningful message, like "Fixed typo", " Added x feature", etc.
  • For your class name, I recommend to use dash to separate two words, instead of doing camelCase. It's a common practice to write HTML class.
<div class="more-details__preview"></div>
  • Remove this development code.
/* ### Primary
- Strong Cyan: hsl(171, 66%, 44%)
- Light Blue: hsl(233, 100%, 69%)
- Dark Grayish Blue: hsl(210, 10%, 33%)
- Grayish Blue: hsl(201, 11%, 66%) */

That's it! Hopefully this is helpful!

Marked as helpful

1

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