3column preview with grid column

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Nice work with this one. Looking at your solution, I would recommend the following for you...

  • Wrap the root content with specific tag like main and attribution with footer tag for readability
  • Clean the whitespaces in the code
  • Add width: 950px; and margin: 2rem auto; in #root rule set
  • Add border to .card-sedans with border-top-left-radius: 0.5rem; and border-bottom-left-radius: 0.5rem;. In .card-luxury with border-top-right-radius: 0.5rem; and border-bottom-right-radius: 0.5rem;
  • Add the hover state of the button with the following concept...

  .button.btn-sedans:hover {
    background: var(--bright-orange);
    border: 1px solid white;
  }
  
  .btn-sedans a:hover {
    color: white;
  }
  

  • In the media queries add the borders to .card-sedans with border-top-left-radius: 0.5rem; and border-top-right-radius: 0.5rem;. In .card-luxury with border-top-right-radius: 0.5rem; and border-top-left-radius: 0.5rem;. Adjust the breakpoints as well.

  Hope this helps and Keep it going!

  Marked as helpful

  0