Skip to main content
Code Review

Return to Answer

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

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : http://stackoverflow.com/a/11143507/7602 https://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is inconsistent, in my mind they should all be surrounded with double quotes.

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : http://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is inconsistent, in my mind they should all be surrounded with double quotes.

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : https://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is inconsistent, in my mind they should all be surrounded with double quotes.

added 2 characters in body
Source Link
konijn
  • 34.2k
  • 5
  • 70
  • 267

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : http://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is consistentinconsistent, in my mind they should all be surrounded with double quotes.

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : http://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is consistent, in my mind they should all be surrounded with double quotes.

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : http://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is inconsistent, in my mind they should all be surrounded with double quotes.

Source Link
konijn
  • 34.2k
  • 5
  • 70
  • 267

From some research:

  • If your code will be used on mobile, I would look into making that phone tag ( which seems to miss </p> ) into a functional phone tag : http://stackoverflow.com/a/11143507/7602

  • I would imagine you want your facebook link to point to your actual page.

  • Your <p>aragraphs are not closed properly..

  • I would leave <p>aragraphs out of <address> and would use styled <span>s

  • frameborder is obsolete for iframe, while iframe should have a title

  • Your quoting of HTML attributes is consistent, in my mind they should all be surrounded with double quotes.

lang-html

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