Social-Proof-Section-Master - HTML5, CSS, Bootstrap Cards

Community feedback

• Account deleted

  Hi,

  Nice one on completing the challenge, it looks really good on desktop.

  • When you expand the window beyond 1440px, everything gets spread and wide, you should make sure to always have a maximum width set to 1440px to avoid this from happening
  • Well, regarding the media queries, first I think 4 is way too much.
  • I think going mobile first naturally gives you the edge... then you'll expand the window and when you think it's big enough to change the layout the you can switch to desktop. This is how I usually approach a challenge, and it works every time.

  Keep coding👍.

  Marked as helpful

  1

  ---

  

  Md Ajmal Shadab•410

  @mdajmalshadabPosted about 3 years ago

  @thulanigamtee Thanks for the suggestion. I'll try to improve my site

  0
• 

  devjakov•210

  @devjakovPosted about 3 years ago

  this page starts to break if I increase width above 1080p

  you should make the container have a max-width of some 1000-1200px i don't know exactly but play around

  What i want to say is that at one point all the content of the website should stop seperating and the width should remain fixed, otherwise the boxes have a ton of space between them on 1440p and 4k screens

  Cheers

  Marked as helpful

  0

  ---

  

  Md Ajmal Shadab•410

  @mdajmalshadabPosted about 3 years ago

  @devjakov thanks! Will try to implement

  0