Social Proof Section

Community feedback

• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted about 3 years ago

  Hey, this is awesome work. I like that you used your own stars! I just have a few things I can add.

  1. I think you should try to use media queries to make your website responsive. I didn't see any in your CSS. But worry not, you can learn those at "https://youtu.be/Xig7NsIE6DI" I hope it's of use. (I low-key think you might have forgotten to add them... if that's the case well,😅)
  2. You didn't have any html issues, it's just the accessibility. You should contain your elements in landmarks like sections, articles... etc. So that it's helpful to those who use screen readers.
  3. Also you can only have on main landmark, consider using a section or article if you want to use another landmark. Otherwise, thanks for this awesome solution. Kudos! Keep on coding!

  Marked as helpful

  2
• 

  Carlos Guzman•285

  @guztrilloPosted about 3 years ago

  Hi Abhishek, avoid using position: absolute and try Flexbox and CSS Grid instead. It will be easier to make the design responsive.

  Marked as helpful

  1
• 

  Bikrant Malla•990

  @vikrantmallaPosted about 3 years ago

  Try to add a grid in the user div It makes easier to position 3 divs and add media queries.

  Marked as helpful

  1
• Account deleted

  You must make it responsive because it doesn't work for almost every width, when you resize things just get on top of each other.

  Marked as helpful

  1