You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- https://codereview.stackexchange.com/a/40262/16414
- https://codereview.stackexchange.com/a/28295/16414
- http://stackoverflow.com/a/11915502/1591669 https://stackoverflow.com/a/11915502/1591669
- http://stackoverflow.com/a/14920215/1591669 https://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.
You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- https://codereview.stackexchange.com/a/40262/16414
- https://codereview.stackexchange.com/a/28295/16414
- http://stackoverflow.com/a/11915502/1591669
- http://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.
You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- https://codereview.stackexchange.com/a/40262/16414
- https://codereview.stackexchange.com/a/28295/16414
- https://stackoverflow.com/a/11915502/1591669
- https://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.
You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- http://codereview.stackexchange.com/a/40262/16414 https://codereview.stackexchange.com/a/40262/16414
- http://codereview.stackexchange.com/a/28295/16414 https://codereview.stackexchange.com/a/28295/16414
- http://stackoverflow.com/a/11915502/1591669
- http://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.
You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- http://codereview.stackexchange.com/a/40262/16414
- http://codereview.stackexchange.com/a/28295/16414
- http://stackoverflow.com/a/11915502/1591669
- http://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.
You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- https://codereview.stackexchange.com/a/40262/16414
- https://codereview.stackexchange.com/a/28295/16414
- http://stackoverflow.com/a/11915502/1591669
- http://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.
You should add the DOCTYPE at the top (before the html
element):
<!DOCTYPE html>
The meta
-charset
should come before the title
(so the encoding is known before parsers reach it):
<meta charset="UTF-8">
<title>Site Title</title>
If this is a page from a whole website, and not just a stand-alone page:
Your site title (which seems to be represented as logo in your case, so the textual title would be the alt
value of that logo) should be a h1
.
The main content (which currently has an h1
), should be in scope of the site title, i.e., h2
. Or even better, use a sectioning content element (section
resp. article
) for your main content. So the structure would look like:
<body>
<h1><img src="logo.png" alt="Site title"></h1> <!-- don’t append "logo" to the alt value -->
<section> <!-- depending on your content, this could also be 'article' -->
<h1>Main content title</h1>
<!-- here comes your whole main content of that page -->
</section>
</body>
You can read more about this in my other answers:
- http://codereview.stackexchange.com/a/40262/16414
- http://codereview.stackexchange.com/a/28295/16414
- http://stackoverflow.com/a/11915502/1591669
- http://stackoverflow.com/a/14920215/1591669
The u
element doesn’t seem to be used correctly here ("unarticulated, though explicitly rendered, non-textual annotation"). If you want to display this text as underlined, use CSS instead:
#listheader h1 {text-decoration: underline;}
But note that in the Web underlines are typically associated with hyperlinks.