3 column preview card

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hey, Aksel! 👋

  Good work on this challenge! 👍

  A few things I'd like to suggest are,

  • Making sure to adjust the border-radius on the card component in the mobile layout so that it only applies to the corners of the entire card (not to the sections, themselves).
  • Adding a max-width to the card component so that it isn't so wide when the layout first changes from desktop to mobile.
  • As palgramming mentioned, making sure the text inside the "Learn More" links doesn't fade out of view upon hover. 😉

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  programmer-shazid•125

  @programmer-shazidPosted over 3 years ago

  @ApplePieGiraffe

  very beautiful advice to boost coding level

  1

  

  Aksel•65

  @akselCoderPosted over 3 years ago

  Hi!! Thanks for the kind advice! I manage to fix the hover part so the text "Learn More" doesn't fade anymore.

  1

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  @akselCoder

  Hey, good work! The text inside the the "Learn More" links is visible, now! 👏

  As for the border-radius, you should use a media query to change where the border-radius is being applied when the screen size decreases. In the mobile layout, you'll want to the border-radius to only be applied to the top corners of the first section and the bottom corners of the bottom section, so you might end up doing something like this,

  .section-1 {
    border-top-left-radius: 5px;
    border-top-right-radius: 5px;
    border-bottom-left-radius: 0px;
    border-bottom-right-radius: 0px;
  }
  
  .section-3 {
    border-radius: 0px 0px 5px 5px; /* this is a shorthand way of writing all of those properties */
  }
  

  I hope that helps! Let me know if you have any more questions. 😉

  1
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  On hover you are losing your "Learn More" text

  1
• 

  Aksel•65

  @akselCoderPosted over 3 years ago

  I know I have a lot to learn but I am struggling within the border-radius on the mobile design.

  0