Huddle landing page using Grid

Community feedback

• 

  T

  Matt Studdert•13,611

  @mattstuddertPosted about 5 years ago

  Nice work Chloe! It looks like you might have your code repo set to private, as it's giving a 404 at the moment. Nice to see you practicing Grid, it's really powerful!

  You could align the social links with the content by moving them over to the middle column, but keeping them on the same row. At least, I hope that's what you were asking for help on. Let me know if it wasn't.

  Also, I'd have a go at making the content on larger screen sizes fit the height of the screen. At the moment, I'm seeing a white area at the bottom of the screen (mine is 1440px wide).

  Keep up the great work! 👍

  0

  ---

  

  Chloe•240

  @cguttwebPosted about 5 years ago

  @mattstuddert Thanks for the reply I have updated the repo link and moved social icons I'm still getting to grips with grid but between it and flexbox I'm sure you can build some pretty cool layouts.

  0
• 

  argel omnes•1,800

  @argelomnesPosted about 5 years ago

  Hi Chloe,

  Good job with Grids! I'm assuming you're pertaining to the sections main-info and social. It's fine not to wrap them in another div and just work on the latter. Like changing grid-column to 2 and removing justify-content.

  0

  ---

  

  argel omnes•1,800

  @argelomnesPosted about 5 years ago

  ps: the link to your repo is broken

  0

  

  Chloe•240

  @cguttwebPosted about 5 years ago

  @argelomnes Whoops thanks have updated the repo link

  0