responsive-type-of-cars-card

Community feedback

• 

  Adriano•38,120

  @AdrianoEscarabotePosted almost 2 years ago

  Hi Burak Durmuşoğlu, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

  I noticed that the images are not appearing, to solve this we can make a simple change in the src of the img:

  <img class="sedan-picture" src="./images/icon-sedans.svg" alt="sedan-image">

  I noticed that the project is not responsive, but we can do something very simple:

  .container {
      width: 100%;
      height: auto;
      /* background-color: gray; */
      display: grid;
      grid-template-columns: 1fr;
      grid-template-rows: 1fr 1fr 1fr;
      margin-top: 40px;
      padding: 20px;
  }
  

  Add this code inside a media query that will trigger on lower resolutions!

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  1

  ---

  

  Burak Durmuşoğlu•160

  @burak2482Posted almost 2 years ago

  @AdrianoEscarabote Thank you so much!

  1

  

  Adriano•38,120

  @AdrianoEscarabotePosted almost 2 years ago

  @burak2482 no problems! this responsiveness code is just a way to do it, there are other ways too, and there are some design details that I didn't fix ahahhah I left it for you to do! an example is border-radius

  Marked as helpful

  1

  

  Burak Durmuşoğlu•160

  @burak2482Posted almost 2 years ago

  @AdrianoEscarabote ahahah i got it!

  1