Having the exact same issue
Just looked at the update log and seen that the Breadcrumbs has been completely rewritten and now use rich snippets – http://support.google.com/webmasters/bin/answer.py?hl=en&answer=185417
I’m all for this and I think it’s a great idea. However, this is going to make it more difficult to customise and it still outputs without using a list. It would be great if there was a setting to allow to output as a list, choose a divider (eg. <span class="divider">/</span>
) and allow html generally (for example putting a home icon for home eg <i class="icon-home"></i> Home
I don’t think there is a work around at the moment for what I do as I can’t even turn rich snippets off (something I’d be reluctant to do, but would make things easier)
I agree, I use(d) HTML in my separators. I think this is pretty critical to keep as a developer—without it limits how we can style the breadcrumbs.
The problem is with /wordpress-seo/frontentd/class-breadcrumbs.php row 238
replace
$output .= esc_html( " $sep " );
with
$output .= " $sep " ;
Thanks, Thomask- works really well. I also had to change the link text and url lines to remove the esc_html
bit:
$link['text'] = $link['text'];
$link['url'] = $link['url'];
I have to say I don’t really like changing the code of plugins over multiple sites- it’s a pain to manage- especially when an update comes along. Hopefully this can be looked into?
i think that yoast will take it as other bug reports and will quickly resolve it. So take it just as a quick fix
Hmm to be honest, I’m wondering why I would or would not allow HTML there… I’ll allow it back in in the next release.
I told you – yoast is a king 😉
Fabulous! Really good news.
I know there are different thoughts on this one, but what’s the thought on making the breadcrumbs outputted as an unordered list or at least giving the option of this?
Any chance this will get remedied soon?
Ricky, what are you referring to? The HTML or it not being coded as a list? I haven’t looked recently, but does the plugin not allow HTML in the breadcrumbs? It’s a shame if it doesn’t. Ideally the plugin should allow more customisation for this.
I can confirm that this escaping issue still exists in v1.4.6.
I had the identical scenario, trying to use Font Awesome to render a Home icon using the HTML <i class="icon-home"></i>
I fixed it by changing line 265 from
$link_output .= '<a href="' . esc_url( $link['url'] ) . '" rel="v:url" property="v:title">' . esc_html( $link['text'] ) . '</a>';
to
$link_output .= '<a href="' . esc_url( $link['url'] ) . '" rel="v:url" property="v:title">' . $link['text'] . '</a>';