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

Flexbox just eased everything!

Tesla_Ambassadorβ€’ 2,980

@tesla-ambassador

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Feel free to check out my code (particularly my CSS) and tell me what you think of my style (organization of my code). That feedback will very much be appreciated.

Community feedback

P
ApplePieGiraffeβ€’ 30,545

@ApplePieGiraffe

Posted

Hey there, Kevin! πŸ‘‹

I'm sorry I'm sooo late! πŸ˜… I finally came to check out your solution and all I can say is well done on this challenge! πŸ‘ You've definitely improved and your solution is looking good! πŸ‘

I just noticed that the card component is a bit wide when the layout first switches from desktop to mobile. You might want to add a max-width to the card in the mobile layout so that it isn't so wide when the layout first changes. πŸ˜‰

I'm looking forward to seeing your solutions to some of the harder challenges! πŸ˜„ And keep on learning JS! πŸ˜†

Bruh, you already knowβ€”keep coding (and happy coding, too)! 😁

Marked as helpful

3

Tesla_Ambassadorβ€’ 2,980

@tesla-ambassador

Posted

@ApplePieGiraffe Hey Bro, thanks for the feedback. I will be sure to add the max-width on the card in the mobile layout. It was looking a bit weird thanks for solving the problem. And am learning JS real hard! I might surprise you with a better challenge one of these days.πŸ˜‰ keep coding (and happy coding, too!)😎

1
Danielle S.β€’ 140

@dstrickl7

Posted

Hi tesla-ambassador! I checked out your CSS code and I think it's logically organized and I think the comments helped with setting up the framework.

Some things I'd like to suggest for the visual aspects of your solution: -If the purple color is pre-defined try using that instead of the default css purple to match the design more closely. -For your images overlay, raise the opacity to around 0.7.

Overall, nice work! Keep coding!

1

Tesla_Ambassadorβ€’ 2,980

@tesla-ambassador

Posted

@dstrickl7 Thank you so much, I have just realized that the purple accent I used is actually lighter than the original one. Thanks a bunch once again.

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