Huddle landing page with single introductory section | CSS Flexbox
Design comparison
Solution retrospective
Any feedback is highly appreciated! Thanks :) !
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- A logo may not be considered as a decoration in HTML because it is often a crucial part of a website's branding and identity, and it may contain important information for the user. you should use an
alt
attribute value with the company name as the text.
- On my computer there is a horizontal scroll that shouldn't be there, let its solution occupy 100% of the screen width, also on much larger screens, its solution has a lot of white space.
-
You should use a CSS reset to remove the default browser styles and make your page look the same in all browsers.
Popular CSS resets:
I hope you find it useful! ๐
Happy coding!
Marked as helpful0 - A logo may not be considered as a decoration in HTML because it is often a crucial part of a website's branding and identity, and it may contain important information for the user. you should use an
- @HassiaiPosted almost 2 years ago
Replace <h2> with <h1> and add the alt attribute
alt=" "
to the img tag to fix the error and accessibility issue. the value of the alt attribute is the description of the image.Give the styling you gave to the main to the body with the exception of display flex and ists properties. For the left and right side space of the page give the main a max-width of 1440px a width of 80% and margin:0 auto.
main{ max-width:1440px; width: 80% margin: 0 auto; }
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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