Skip to content
Submitted about 1 month ago

display flex

JavohirXolmamatov•140
@JavohirXolmamatov
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social links profile
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

great

What challenges did you encounter, and how did you overcome them?

great

What specific areas of your project would you like help with?

great

Community feedback

  • Daniel•260
    @FlashdanielPosted about 1 month ago

    Great Job! I like to point out that you should learn to Make use of Accessibility Landmarks like <header>, <main>, <footer>. Always put yourstyles in a different file separate form the HTML files. Make proper use of the heading tags. Always start with h1 then h2, h3 and so on. You can use css to size then to your preferred. I personally do everything possible to avoid using the css height property in my code. And if you must use height avoid using the vh unit. Do this

    * {
box-sizing: border-box;

    This applies it to all the elements. Do put it on the body as it is not inherited. Cheers!

    Marked as helpful
    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord