@Alex-Archer-I
Posted
This is a list of the links, so it's better to use ul tag with li here =) Something like this:
<ul>
<li><a href="#">I'm a link!</a></li>
<li><a href="#">I'm a link!</a></li>
<li><a href="#">I'm a link!</a></li>
<li><a href="#">I'm a link!</a></li>
<li><a href="#">I'm a link!</a></li>
</ul>
And I recommend you to use min-height for the body tag in case if the content will be lager than user's screen.
By the way, that's cool that you pay a lot of attention to details =)
Marked as helpful
@Alex-Archer-I
Thank you! You're right, I didn't even think about using a list for the links. That would have made much more sense.
Should I use min-width for the body as well instead of the fixed width: 100vw;?
@Alex-Archer-I
Posted
@AStombaugh
That isn't really necessary unless you assume that page will have content which could be wider that the screen. But that is very rare case and it's considered bad practice if the user have to scroll horizontally (that's why we are using media queries, responsive values and stuff).
Even more, you can not to set width for body at all as it equal screen's width by default. That could be said about height: auto and flex-direction: row - these are default values and you can freely omit them.
Marked as helpful
