@DarkstarXDD
Posted
- There is no need to specify a
min-width
on your container. No need to specify awidth
either.max-width
is enough, could be better if it was inrem
instead ofpx
. - No need to specify a
width
for the social media links either. I see you have given them a width of 95%. Avoid using fixed widths on text elements. - There is no
<header>
in this design.<header>
purpose is to hold primary repeating content across every page of a site, such as the logo of the brand and the navigation bar. - The social media links should be list items. So each
<a>
should be inside a<li>
. - There is no reason to have a
<div>
inside the<a>
. You can make the<a>
adisplay: block
and put the text directly inside the<a>
. - Would be nice if you give your page a title. Currently it says "Document". I usually give the challenge name as the title.
Marked as helpful
@Ridwan10000
Posted
@DarkstarXDD Thanks for your suggestion. I will be careful while designing from next on.