@PhoenixDev22
Posted
Greeting @goldenekpendu,
I have few suggestions regarding your solution:
-
Use <main> to wrap the body content
<main class="container">
) and afooter
(which will be the attribution).<Footer>
should be outside the`` <main >` . HTML5 landmark elements are used to improve navigation . -
To tackle the accessibility issues, you can add a
<h1>
withclass="sr-only"
(Hidden visually, but present for assistive tech).
.sr-only {
border: 0 !important;
clip: rect(1px, 1px, 1px, 1px) !important;
-webkit-clip-path: inset(50%) !important;
clip-path: inset(50%) !important;
height: 1px !important;
margin: -1px !important;
overflow: hidden !important;
padding: 0 !important;
position: absolute !important;
width: 1px !important;
white-space: nowrap !important;
}
-
Don't use <div> for meningful content .
-
min-height: 100vh;/using vh (viewport height) units to allow the body to set a minimum height value based upon the full height of the viewport.This also allows the body to to grow taller if the content outgrows the visible page./
-
width: 350px;
an explicit width is not a good way . Remove the width from the main component and change it tomax width
instead. That will let it shrink a little when it needs to. -
You should use
em
andrem
units .Bothem
andrem
are flexible, Usingpx
won't allow the user to control the font size based on their needs.
Overall , your solution is good. Hopefully this feedback helps.
Marked as helpful
@goldenekpendu
Posted
@PhoenixDev22 thanks a lot for your indepth feedback. I'll make the changes. Thanks again