Design comparison
SolutionDesign
Community feedback
- @polukarpPosted about 2 years ago
Love your solution, but I'd suggest a couple of minor fixes here:
- Why is your
<div className="nav-bottom">
not a part of your<nav>
element but still a part of your Header component? Try placing it in your main content, especially as it contains the<h1>
tag. You can name it a hero section if you wish. - Also your
<ul>
elements in the footer should not contain directly any element rather than<li>
,<template>
elements.
Marked as helpful0@bekzodturgunoffPosted about 2 years ago@polukarp Thanks for commenting to my solution. I'll try to fix them.
0 - Why is your
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