Flexbox just eased everything!

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  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•3,000

  @tesla-ambassadorPosted over 3 years ago

  @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

  @dstrickl7Posted over 3 years ago

  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•3,000

  @tesla-ambassadorPosted over 3 years ago

  @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