Google launches from not long time, a better semantic way to write breadcrumb in html.
It will help google(and other for sure) to better detect and understand content of breadcrumbs.

<div itemscope itemtype="http://data-vocabulary.org/Breadcrumb">
  <a href="http://www.example.com/dresses" itemprop="url">
    <span itemprop="title">Dresses</span>
  </a> ›
</div>  
<div itemscope itemtype="http://data-vocabulary.org/Breadcrumb">
  <a href="http://www.example.com/dresses/real" itemprop="url">
    <span itemprop="title">Real Dresses</span>
  </a> ›
</div>  
<div itemscope itemtype="http://data-vocabulary.org/Breadcrumb">
  <a href="http://www.example.com/clothes/dresses/real/green" itemprop="url">
    <span itemprop="title">Real Green Dresses</span>
  </a>
</div>

I make a patch for 6.x-1.x to display microdata during menu_breadcrumb generation.
I add also a settings page to choose if microdata have to be generate or not.

This patch works if you have only one breadcrumb on the page. For multiple breadcrumbs, microdata syntax is different.

I'll attach patch in next comment to have issue number

CommentFileSizeAuthor
#1 microdata-compliancy-1334400-1.patch5.44 KBgoz

Comments

goz’s picture

goz commented
StatusFileSize
new microdata-compliancy-1334400-1.patch5.44 KB

Here is the patch (also commited in a branch on git)

goz’s picture

goz commented
Status: Active » Needs review

Someone to review this patch please ?

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Status: Needs review » Closed (works as designed)

Thanks GoZ. But I'm afraid I don't see why this is a patch against menu_breadcrumb. Can it not be implemented in the theme layer entirely?

Microdata support sounds great, but I'd think it should either be a separate module (so sites not using menu_breadcrumb can take advantage of it) or implemented in the theme.

I'm happy to hear your thoughts if you do believe this needs to be implemented in this module. Until convinced otherwise, I'm marking this "works as designed".

goz’s picture

goz commented

I also think this feature should be in a theme layer.
Unfortunately, drupal 6 doesn't permit to make this once you generate your $breadcrumb[] array (with markup link for example)
On theme_breadcrumb function, we only take $breadcrumbs markup items. The only way to make microdata structure describe on main code example issue at this state is to use regexp (i hope noone do that).

Instead of use this patch (and to make the microdata implementation by theme layer), we could add a theme_menu_breadcrumb_item (overridable by preprocess & .tpl) and let themers override breadcrumb item if they want.

For example, actually we make a breadcrumb item :

$breadcrumb[] = $is_front ? l(t('Home'), '<front>') : l(drupal_get_title(), $_GET['q'], array('html' => TRUE,));

So we can make now :

$breadcrumb[] = $is_front ? theme('menu_breadcrumb_item', array(t('Home'), url('<front>')) : theme('menu_breadcrumb_item', array(drupal_get_title(), $_GET['q'], array('html' => TRUE,)));

Where theme_menu_breadcrumb_item take title, url, options arguments.

In this case, themer can easily override item with mytheme_preprocess_menu_breadcrumb_item or menu_breadcrumb_item.tpl.php to put his microdata markup.

Does this way is better for you ?

goz’s picture

goz commented

Since D7 doesn't improve $breadcrumb to be free of tags, could we work on a solution which will integrate microdata in breadcrumb modules (like describe on #4) ?

I don't think using regexp on template preprocess is a good solution to add microdata compliancy (and it's the only solution for the moment if we don't want to patch your module).