Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive landing page with flexbox and media queries

Nikola 190

@porumbachanov


Design comparison


SolutionDesign

Community feedback

Alex 3,130

@Alex-Archer-I

Posted

Hi!

Congrats with new challenge. That's a cool one =)

Here is a couple of unasked advices =)

You used landmark tags, like header and footer, that's cool. But the header shouldn't contain h1 tag. You see, when the header and the footer are on top level, they are special parts of the site which remains on every page. So, in the most cases header includes logo and nav.

For the phone number and email in the footer you can use special links:

<a href="mailto:[email protected]">[email protected]</a>
<a href="tel:+1-543-123-4567">+1-543-123-4567</a>

The mailto link will open the mail agent on click, and the tel one will open the phone app if user tap it on their phone. So it's kinda good user experience to use those links and good for accessibility. Also you can wrap them into address tag.

Hope that helps =) Keep doing =)

Marked as helpful

1

Nikola 190

@porumbachanov

Posted

@Alex-Archer-I Makes sense, thanks for the advice! I'll make the changes and keep it in mind going forward.

1

Please log in to post a comment

Log in with GitHub
Discord logo

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