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
Request path contains unescaped characters
Not Found
Not Found
Not Found

Submitted

Mobile first approach using FlexBox

Rita 30

@ritakerkovits

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


Just finished another challenge where I utilized the FlexBox technique! Great task to practice flexbox properties like the flex-basis.:)

Community feedback

Venus 1,640

@VenusY

Posted

Great job on this challenge! You've done really well making it look like the design, and it's responsive too, which is good.

There are some minor tweaks that you could make, however.

I noticed that when hovering over the button, the card shifts a bit.

I believe this is happening because you removed the default border from the button, but then you add a border when the user interacts with the button. The added height of the border causes the card's height to increase as well.

You can fix this by simply moving the border style to the button:

button {
  border: 1px solid hsl(0, 0%, 95%);
}

Since the border colour is the same as the button's background colour when it's not being interacted with, it blends in, so you can't tell that there's a border there.

To make it even more obvious that it's a clickable object, you could change the cursor type when the user hovers over a button:

button {
  cursor: pointer;
}

I also noticed that you manually set each button's background colour to that of its respective card, but since they're the same, you can just set it to its default value, which is transparent:

button:hover,
button:focus-visible {
  background-colour: transparent;
}

Finally, this is a very minor thing, but I noticed the slight typo of 'SUVs' instead of 'SUVS' in the middle card.

Other than that, this is a really good solution!

Hope this has been helpful! :)

Marked as helpful

1

Rita 30

@ritakerkovits

Posted

Hi @VenusY

I really appreciate your feedback, huge thanks for that!

Indeed, the card shifts while hovering, and to be honest, I was a bit stuck because of it. So, these are great pieces of advice. I will apply them to my solution :)

1
Venus 1,640

@VenusY

Posted

@ritakerkovits No worries! I'm glad I managed to help you out a little. Feel free to give me another reply if you get stuck on anything, and I'll try my best to help you out.

I would appreciate it if you marked my comment 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