The markup around the breadcrumb needs to be updated for HTML5. We should try to follow what's going on in D8 core.

See #1347266: Convert theme_breadcrumb() to HTML5

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JohnAlbin’s picture

Title: Convert breadcrumb to HTML » Convert breadcrumb markup to HTML
Status: Active » Fixed
JohnAlbin’s picture

Status: Fixed » Postponed

We'll still need to track #1347266: Convert theme_breadcrumb() to HTML5 to see if the final patch changes the markup.

JohnAlbin’s picture

Title: Convert breadcrumb markup to HTML » Convert breadcrumb markup to HTML5

Not sure how I missed the 5 on the end of the issue title. :-p

maartenverbaarschot’s picture

Status: Postponed » Needs review
FileSize
747 bytes

It became an <ol> :)

JohnAlbin’s picture

Status: Needs review » Fixed

Yep. Just noticed that.

Thanks, Maarten! You've got a commit now. :-) http://drupal.org/user/1305466

echoz’s picture

Status: Fixed » Needs review
FileSize
665 bytes

..and the css

JohnAlbin’s picture

Component: CSS/HTML markup » layout.css
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.