summaryrefslogtreecommitdiff
path: root/includes/theme.inc
diff options
context:
space:
mode:
authorwebchick <webchick@24967.no-reply.drupal.org>2012-04-28 23:44:44 -0700
committerwebchick <webchick@24967.no-reply.drupal.org>2012-04-28 23:44:44 -0700
commit53ff0485deef33ba83fe9a4c841c2f2f91160f46 (patch)
tree6c49ba06c778709a70be36c2a0003bcadf4d4972 /includes/theme.inc
parentfaf3e8575e043ae8658f7237b196dd0b83ab1a39 (diff)
downloadbrdo-53ff0485deef33ba83fe9a4c841c2f2f91160f46.tar.gz
brdo-53ff0485deef33ba83fe9a4c841c2f2f91160f46.tar.bz2
Issue #1349722 by kgoel, Everett Zufelt, xjm: Fixed Who's new & Who's online blocks have empty headings.
Diffstat (limited to 'includes/theme.inc')
-rw-r--r--includes/theme.inc5
1 files changed, 4 insertions, 1 deletions
diff --git a/includes/theme.inc b/includes/theme.inc
index 0712f8797..268795e17 100644
--- a/includes/theme.inc
+++ b/includes/theme.inc
@@ -1967,8 +1967,11 @@ function theme_item_list($variables) {
$type = $variables['type'];
$attributes = $variables['attributes'];
+ // Only output the list container and title, if there are any list items.
+ // Check to see whether the block title exists before adding a header.
+ // Empty headers are not semantic and present accessibility challenges.
$output = '<div class="item-list">';
- if (isset($title)) {
+ if (isset($title) && $title !== '') {
$output .= '<h3>' . $title . '</h3>';
}