Huddle Landing Page using Flexbox

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted over 4 years ago

  Hi, Guliye! 👋

  Good job on this challenge! 👏

  I suggest,

  • Adding a maximum width to the main container or wrapper to prevent the content of the page from becoming too stretched on extra-large screens.
  • Horizontally centering the hero image in the mobile layout of the site.
  • Adding min-height: 100vh to the body to ensure that it is always as large as the viewport and the background image fills the entire area of the screen.

  Keep coding (and happy coding, too)! 😁

  11
• 

  Jane•1,040

  @janegca

  Posted over 4 years ago

  Hi Guliye, for responsive design check out Kevin Powell's channel on YouTube and his Conquering Responsive Layouts course. Or start with Free Code Camp's Responsive Web Design

  BEM (Block-Element-Modifier) is a methodology for naming classes in CSS, for more info checkout this BEM Naming Cheat Sheet

  Hope that's of some help. Happy coding.

  11
• 

  Jane•1,040

  @janegca

  Posted over 4 years ago

  HI Gullye, your design is nicely responsive. Two small nits, the text on the button doesn't match the design; the hover effect is missing on the social icons (checkout CSS-Tricks: Change Color of SVG on Hover

  Nice job overall. Happy coding.

  11