Closed (fixed)
Project:
Skeleton
Version:
7.x-1.0
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
22 Jun 2012 at 18:47 UTC
Updated:
4 Jan 2014 at 02:05 UTC
Jump to comment: Most recent
The skeletontheme_breadcrumb() function in template.php outputs an HTML5 nav tag while the rest of the theme utilizes XHTML+RDFa 1.0. This function should use div instead.
function skeletontheme_breadcrumb($variables) {
$breadcrumb = $variables['breadcrumb'];
if (!empty($breadcrumb)) {
// Use CSS to hide titile .element-invisible.
$output = '<h2 class="element-invisible">' . t('You are here') . '</h2>';
// comment below line to hide current page to breadcrumb
$breadcrumb[] = drupal_get_title();
$output .= '<nav class="breadcrumb">' . implode('<span class="sep">»</span>', $breadcrumb) . '</nav>';
return $output;
}
}
On a less critical note, »should be used in place of a literal right angle quote symbol.
Comments
Comment #1
gtsopour commentedComment #2
gtsopour commentedThank you aberger for this report. I will examine this issue directly.
Thanks
George
Comment #3
gtsopour commentedJust made and committed this change http://drupalcode.org/project/skeletontheme.git/commit/daf10d1
Thanks
George
Comment #4.0
(not verified) commentedfixed a mistake