Responsive Mobile First Huddle Landing Page with CSS Flexbox and Grid
Design comparison
Solution retrospective
I have tried using variables extensively for the first time to define paddings and all sort of spacings. I would like your feedback on that! Also any feedback to DRY the code and organise it efficiently is welcome! Thanks in advance.
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, awesome work on this one. Layout in desktop looks fine, just some text needs to be bigger so that some white-spaces could be occupied. Mobile layout looks great.
Well, I don't really used plain css, I use scss so that I could have different module/files, this way, I won't be using a single file it will prevent it from being cluttered. Have a try using scss and search about 7-1 pattern.
Some other suggestions would be:
- You don't really need a
nav
on theheader
since there are no navigational links that are used, adiv
would be fine. - I think the
try for free
and any other buttons that looks similar to that is better when it usesa
tags, because it looks more likely to be a page link for you to try it. - On the
img
in the hero-section, if you are usingaria-hidden="true"
on the image, then use an emptyalt
as well likealt=""
on it, since you are hiding the image. - Also, for the icons below the hero-sections, the icons that are used by the numbers should have
alt=""
. - The text
1.4k+
and the2.7m+
isn't really suited to be a heading tag, since it doesn't add any extra information about the section where it is used. When a design has bold text, it doesn't always mean that they are heading tags. - Sam for the other images,
alt=""
if it is just decorative.
FOOTER
- The
alt
value for the telephone and email icon should be emptyalt=""
and needaria-hidden="true"
. Decorative images should be hidden. - The social media links could be nested inside
ul
element, since those are "list" of links. - The form's
input
needs to have alabel
associated with it, use ansr-only
to thelabel
you are going to used, describe the purpose of theinput
. - Lastly, just to practice more js, create a email validation.
Aside from those, great work again on this one.
Marked as helpful2 - You don't really need a
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