Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In the interest of including microdata for optimizing the display of goo... #1228

Closed
wants to merge 1 commit into from

Conversation

BHEADRICK
Copy link

...gle rich snippets, this updated breadcrumb layout will cause the breadcrumbs to be included in the google rich snippet, so long as the default wap_before is changed from

to

…google rich snippets, this updated breadcrumb layout will cause the breadcrumbs to be included in the google rich snippet, so long as the default wap_before is changed from <div id="breadcrumb"> to <div id="breadcrumb" itemscope itemtype="http://data-vocabulary.org/Breadcrumb">
@BHEADRICK
Copy link
Author

This is actually RDFa, but I tried using Microdata instead of RDFa, and the Rich Snippets Testing Tool only picked up on RDFa and not Microdata for breadcrumbs

@BHEADRICK BHEADRICK closed this Jul 6, 2012
@BHEADRICK BHEADRICK reopened this Jul 6, 2012
@mikejolley
Copy link
Member

I think we should keep it consistant and use microdata/schema.org.

Breadcrumbs require a webpage item,

itemscope itemtype="http://schema.org/WebPage"

on the body tag. Ill add the breadcrumb microdata but in order to use it the theme will need to add this. We can't access the body property.

@mikejolley mikejolley closed this Jul 10, 2012
mikejolley added a commit that referenced this pull request Jul 10, 2012
@BHEADRICK
Copy link
Author

That was my first thought, but when testing with google's rich snippet
tool, microdata formatted breadcrumbs don't show up even though Google says
they should.
Thanks,

Bryan Headrick
http://www.bryanheadrick.com

On Tue, Jul 10, 2012 at 12:18 PM, Mike Jolley <
reply@reply.github.com

wrote:

I think we should keep it consistant and use microdata/schema.org.

Breadcrumbs require a webpage item,

itemscope itemtype="http://schema.org/WebPage"

on the body tag. Ill add the breadcrumb microdata but in order to use it
the theme will need to add this. We can't access the body property.


Reply to this email directly or view it on GitHub:
#1228 (comment)

@mikejolley
Copy link
Member

Once you add the body WebPage itemtype, they are picked up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants