Responsive Website Using CSS Grid And Flex

Solution retrospective
Please provide feedback
Please log in to post a comment
Log in with GitHubCommunity feedback
- @pikapikamart
Hey, great work on this one. The layout both desktop and mobile is good and the responsiveness as well.
Some suggestions would be:
- The company logo's
alt
text should have beenalt="Fylo"
since the logo itself have that word, better to use it in thealt
. Also, when usingalt
attribute, do not include words like "icon" "image" "logo" or anything else that relates to "graphic", assistive tech will handle those for you. - The company logo in the header as well, since you are adding an interaction with it, better to nest it inside
a
tag, along with anaria-label="homepage"
, since mostly company logo links to the homepage. - Have a
main
element that wraps the whole main content. This will clear up the issue in your report as well as adding more to the structure of your website. - Do not skip a heading level. Use heading tags incrementally by one. If you use
h1
make sure to haveh2
before you useh3
. On your solution, you didn't useh3
but you usedh4
andh5
. - The
alt
text for the person in the testimonial could have used their name as the value, likealt="Satish Patel"
. - Only use 1
h1
per page. You use the 1sth1
on the hero-section, the 2nd one is at the cta-section. - On the submit button in the form, it would be better to use
button type="submit"
, the reason is that, you could add a more descriptive information about what thatbutton
does, you could addaria-label="submit form"
on thebutton
instead of justget started for free
which doesn't really describe what it does right. - Company logo in the
footer
could have usedalt="fylo"
as well. - The links on the
footer
could have usedul
as well, since those are list of links. - The
a
tag that holds the social media icons should have anaria-label
, likearia-label="facebook"
for thea
tag that wraps the facebook icon. This will ensure that screen reader users knows where this link should go.
Aside from those, great work.
- The company logo's
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