Huddle Landing Page With Alternating Blocks

Community feedback

• 

  argel omnes•1,800

  @argelomnesPosted about 5 years ago

  Hey Blaise, you can use width: 100% to your images so they automatically scale up.

  1

  ---

  

  Blaise•20

  @afrojac10Posted about 5 years ago

  @argelomnes Wow thanks for the advice

  0
• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 5 years ago

  Hey Blaise, great work on this challenge and I'm really liking the detailed README.md for more information!

  With regards to the feature items not displaying correctly for desktop, I'd recommend wrapping the text content inside a div for each item. This way the img and the div will be siblings and then could be stacked on mobile and put side-by-side for desktop.

  For the footer on desktop, it seems you don't currently have any styles to change the layout for desktop, so I'd start adding those styles inside a media query. You'll see that if you add display: flex; to the .footer class everything jumps up to the same row. It obviously doesn't look right but hopefully, that will get you started.

  Here's a great Udacity course on building responsive sites if you want to learn more about it.

  As a quick side note, I saw that you're using basically the same styles 3x over for the social icons. You could instead target the .fab class and write those styles just once.

  Let me know if you have any questions, keep up the great work!

  1

  ---

  

  Blaise•20

  @afrojac10Posted about 5 years ago

  Will make the changes, thanks for the review and feedback.

  0
• 

  Blaise•20

  @afrojac10Posted about 5 years ago

  Fixed the Hero Section and Footer view for desktop.

  Fix only changed Desktop view as site already optimized on mobile.

  0