Responsive Social Proof Section

Community feedback

• 

  Jose Luis Flores Vacio•210

  @JoseLuisFVPosted over 2 years ago

  Hi, I saw your solution, for background image, i recommend the css property background to images because in those cases it is only decorative

  body{
   background: 
          url('../images/bg-pattern-top-desktop.svg'),
          url('../images/bg-pattern-bottom-desktop.svg');
    background-repeat: no-repeat, no-repeat;
    background-position: top left, bottom right;
  }
  

  For mobile view, the reason is that you have double scrollbar, and i don't know what is the cause.

  This is my repo This is my live.

  But your work is great!, dont get down, you can do it, sorry this is not my native language

  Marked as helpful

  1

  ---

  

  Soojeong•220

  @kongguksuPosted over 2 years ago

  @JoseLuisFV Thank you for your example code! I'll try this out. Thanks for the encouragement as well :)

  0
• 

  Brandon•290

  @branalex94Posted over 2 years ago

  Hello there! I hope you are well. I'm here hoping my advice can help somehow. I've checked your live site and it seems you are using a div to set the background images. I'd recommend investigating the "background" CSS property because with it you can apply multiple images at once and it may be more convenient that way.

  By having that div, it seems like it's ocuppying some space it should'nt and you end up having double scrolling on mobile.

  To use different images depending on size I'd recommend you to use media queries as well. Just check the background property and it may help a lot. Hope i could be of help! If it may be useful, you can check my solution. It's not perfect though, but it may give you some answers.

  Marked as helpful

  1

  ---

  

  Soojeong•220

  @kongguksuPosted over 2 years ago

  @branalex94 I'll try using the background property to fix the scrolling issue. I'll definitely check out your solution too. Thank you for your feedback!

  1