HTML/CSS (error to be fixed)

Community feedback

• 

  spyder•315

  @ratan17Posted over 3 years ago

  you have used vh units everywhere, unnecessary divs to wrap content. avoid setting height use instead padding property with rem values eg ) button{ padding: 1rem; }

  Marked as helpful

  2

  ---

  

  Lucky•30

  @anupamthe99Posted over 3 years ago

  @ratan17 thank you

  0
• 

  vishal wankhade•495

  @Vishal-wankhadePosted over 3 years ago

  give the google font link in the html file instead of import , you did not add the given font-family, font-weight to the text ,button bg-color is not white your site is not looking well in mid screen devices hope this is helpful

  Marked as helpful

  0

  ---

  

  Lucky•30

  @anupamthe99Posted over 3 years ago

  @Vishal-wankhade thanks for your suggestion i will fixed it .

  0
• 

  Ahmed Faisal•5,095

  @afrusselPosted over 3 years ago

  Button is cutting off for big screen. In mobile view boxes are not fit. There is a horizontal scroll here

  Marked as helpful

  0

  ---

  

  Lucky•30

  @anupamthe99Posted over 3 years ago

  @afrussel thanks for your feedback

  0
• 

  Hafizan Adli•1,190

  @hafizanadliPosted over 3 years ago

  Hello, Here are my suggestions

  • Wrap the suvs, sedans and luxury with a container (e.g., main tag). Set the body min-height to 100vh (use min-height instead of fix height).And you can easily centering the main with flex in body.
  • You don't need to wrap img with div
  • For text you can use p tag instead of div

  Great Job!

  0

  ---

  

  Lucky•30

  @anupamthe99Posted over 3 years ago

  @hafizanadli thanks a lot .Your feedback really help a lot .

  1