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

CSS Flexbox

Sam Kikoma 140

@Sam-Kikoma

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

Solution retrospective


Any feedback is welcome

Community feedback

Travolgi 🍕 31,420

@denielden

Posted

Hey Sam, congratulations on completing the challenge! You did a great job 😉

Let me give you some little tips for optimizing your code:

  • Tip of graphic design: with font-family:" Big Shoulders Display ", cursive the browser will use the Comics Sans font when it doesn't find the first font indicated (you can seen during loading)... for the designer it's a really awful font! I would rather replace it with a font-family:" Big Shoulders Display ", sans-serif much more similar to the primary font.
  • remove width: 100% from main tag because it's superfluous
  • it also assigns the same border to the normal state of the buttons so that when you hover over them you don't see that leap in the design
  • add transition on the element with hover effect

Hope this help! Happy coding 😁

Marked as helpful

1

Sam Kikoma 140

@Sam-Kikoma

Posted

@denielden Appreciate the feedback. I'll definitely look into optimizing my code

1
Divine Obeten 2,415

@Deevyn9

Posted

Hi, Great representation, adjusting the width of the buttons would be gret. Happy coding

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