Skip to main content
Code Review

Return to Answer

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

Your Microdata is fine.

Some HTML suggestions:

Your Microdata is fine.

Some HTML suggestions:

  • I wouldn’t use p as container for iframe + the map. I’d use div instead.

  • Don’t use <br> after the iframe (br is for line breaks that are part of the content only). Your usage of br in the postal address is correct.

  • Don’t use small for the link to the map (as it doesn’t seem to be a side comment).

  • You could use rel-external for the external links to Google and Facebook.

  • You could link the telephone number with the tel URI scheme. But then you should include the anchor text in a span element, as schema.org’s telephone property expects "Text", not "URL" (which would be the value for the itemprop if used on a).

     <a href="tel:555..."><span itemprop=telephone>555 555-5555</span></a>
    
  • In contrast to what tomdemuyt answered, using p for the postal address seems to be fine (as it’s used in an example in the HTML5 spec).

Your Microdata is fine.

Some HTML suggestions:

  • I wouldn’t use p as container for iframe + the map. I’d use div instead.

  • Don’t use <br> after the iframe (br is for line breaks that are part of the content only). Your usage of br in the postal address is correct.

  • Don’t use small for the link to the map (as it doesn’t seem to be a side comment).

  • You could use rel-external for the external links to Google and Facebook.

  • You could link the telephone number with the tel URI scheme. But then you should include the anchor text in a span element, as schema.org’s telephone property expects "Text", not "URL" (which would be the value for the itemprop if used on a).

     <a href="tel:555..."><span itemprop=telephone>555 555-5555</span></a>
    
  • In contrast to what tomdemuyt answered, using p for the postal address seems to be fine (as it’s used in an example in the HTML5 spec).

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

Your Microdata is fine.

Some HTML suggestions:

Your Microdata is fine.

Some HTML suggestions:

  • I wouldn’t use p as container for iframe + the map. I’d use div instead.

  • Don’t use <br> after the iframe (br is for line breaks that are part of the content only). Your usage of br in the postal address is correct.

  • Don’t use small for the link to the map (as it doesn’t seem to be a side comment).

  • You could use rel-external for the external links to Google and Facebook.

  • You could link the telephone number with the tel URI scheme. But then you should include the anchor text in a span element, as schema.org’s telephone property expects "Text", not "URL" (which would be the value for the itemprop if used on a).

     <a href="tel:555..."><span itemprop=telephone>555 555-5555</span></a>
    
  • In contrast to what tomdemuyt answered, using p for the postal address seems to be fine (as it’s used in an example in the HTML5 spec).

Your Microdata is fine.

Some HTML suggestions:

  • I wouldn’t use p as container for iframe + the map. I’d use div instead.

  • Don’t use <br> after the iframe (br is for line breaks that are part of the content only). Your usage of br in the postal address is correct.

  • Don’t use small for the link to the map (as it doesn’t seem to be a side comment).

  • You could use rel-external for the external links to Google and Facebook.

  • You could link the telephone number with the tel URI scheme. But then you should include the anchor text in a span element, as schema.org’s telephone property expects "Text", not "URL" (which would be the value for the itemprop if used on a).

     <a href="tel:555..."><span itemprop=telephone>555 555-5555</span></a>
    
  • In contrast to what tomdemuyt answered, using p for the postal address seems to be fine (as it’s used in an example in the HTML5 spec).

Source Link
unor
  • 2.7k
  • 15
  • 24

Your Microdata is fine.

Some HTML suggestions:

  • I wouldn’t use p as container for iframe + the map. I’d use div instead.

  • Don’t use <br> after the iframe (br is for line breaks that are part of the content only). Your usage of br in the postal address is correct.

  • Don’t use small for the link to the map (as it doesn’t seem to be a side comment).

  • You could use rel-external for the external links to Google and Facebook.

  • You could link the telephone number with the tel URI scheme. But then you should include the anchor text in a span element, as schema.org’s telephone property expects "Text", not "URL" (which would be the value for the itemprop if used on a).

     <a href="tel:555..."><span itemprop=telephone>555 555-5555</span></a>
    
  • In contrast to what tomdemuyt answered, using p for the postal address seems to be fine (as it’s used in an example in the HTML5 spec).

lang-html

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