Skip to main content
Code Review

Return to Answer

replaced http://stackoverflow.com/ with https://stackoverflow.com/
Source Link

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:


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:


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:


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.

replaced http://codereview.stackexchange.com/ with https://codereview.stackexchange.com/
Source Link

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:


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:


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:


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.

Source Link
unor
  • 2.7k
  • 15
  • 24

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:


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.

default

AltStyle によって変換されたページ (->オリジナル) /