huddle-landing-page-with-alternating-feature-blocks html CSS Flexbox

Community feedback

• 

  Muhammad Shajjar•1,100

  @muhammadshajjarPosted about 3 years ago

  Hey congratulations! on completing your first junior level challenge, it was always fun to complete the challenge, Your overall layout looks good but try to add some more things to it for better design and code

  • Avoid too much classes inheritance because sometimes it would cause specificity issues header .header_content .build_community instead of this you may go with .build_community

  • Try to add max-width to your body or make a container class that wraps your all content, by adding it you can control your content from growing too much on larger viewports

  • Your logo is an image so font-size would not work in this case use height or width instead, And try to fix it, it looks too big on small viewports

  • Create your link to mailto link, it is used to redirect to an email address instead of a web page URL. To create a Mailto link, you need to insert "mailto:" parameter in href, like the following: <a href="mailto:[email protected]">[email protected]</a> And for the mobile number you can use tel link it is used to redirect to a phone dialler directly, like the following, <a href=“tel:+1-543-123-4567”>+1-543-123-4567</a>

  • Give your cards some shadow so they would look like cards

  Marked as helpful

  0

  ---

  

  Philemon Bahati•580

  @bahati7Posted about 3 years ago

  @muhammadshajjar thank you so much for your helpfull feedback. I will try to apply them.

  0