Skip to content

Commit 0f2d9ba

Browse files
Merge pull request #2359 from rhamilto/sg00dwin-vert-nav-collapsed-icon-title
Automatic merge from submit-queue. Add alt text for left nav in mini-mode Fixes #2294 Replaces #2309 fyi @sg00dwin
2 parents ac7501d + ce487ec commit 0f2d9ba

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

Diff for: app/views/_sidebar.html

+6-3
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,17 @@
2323
ng-mouseenter="onMouseEnter(primaryItem)"
2424
ng-mouseleave="onMouseLeave(primaryItem)"
2525
class="list-group-item">
26-
<a ng-if="primaryItem.href" ng-href="{{navURL(primaryItem.href)}}" ng-click="itemClicked(primaryItem)">
27-
<span class="{{primaryItem.iconClass}}"></span> <span class="list-group-item-value">{{primaryItem.label}}</span>
26+
<a
27+
ng-if="primaryItem.href"
28+
ng-href="{{navURL(primaryItem.href)}}"
29+
ng-click="itemClicked(primaryItem)">
30+
<span title="{{primaryItem.label}}" class="{{primaryItem.iconClass}}" aria-hidden="true"></span> <span class="list-group-item-value">{{primaryItem.label}}</span> <span ng-if="nav.collapsed && !isMobile" class="sr-only">{{primaryItem.label}}</span>
2831
</a>
2932
<a
3033
ng-if="!primaryItem.href"
3134
href=""
3235
ng-click="itemClicked(primaryItem)">
33-
<span class="{{primaryItem.iconClass}}"></span> <span class="list-group-item-value">{{primaryItem.label}}</span>
36+
<span title="{{primaryItem.label}}" class="{{primaryItem.iconClass}}" aria-hidden="true"></span> <span class="list-group-item-value">{{primaryItem.label}}</span> <span ng-if="nav.collapsed && !isMobile" class="sr-only">{{primaryItem.label}}</span>
3437
</a>
3538

3639
<!-- Secondary Nav -->

Diff for: dist/scripts/templates.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -490,10 +490,10 @@ angular.module('openshiftConsoleTemplates', []).run(['$templateCache', function(
490490
" 'secondary-nav-item-pf': primaryItem.secondaryNavSections.length\n" +
491491
" }\" ng-if=\"show(primaryItem)\" ng-mouseenter=\"onMouseEnter(primaryItem)\" ng-mouseleave=\"onMouseLeave(primaryItem)\" class=\"list-group-item\">\n" +
492492
"<a ng-if=\"primaryItem.href\" ng-href=\"{{navURL(primaryItem.href)}}\" ng-click=\"itemClicked(primaryItem)\">\n" +
493-
"<span class=\"{{primaryItem.iconClass}}\"></span> <span class=\"list-group-item-value\">{{primaryItem.label}}</span>\n" +
493+
"<span title=\"{{primaryItem.label}}\" class=\"{{primaryItem.iconClass}}\" aria-hidden=\"true\"></span> <span class=\"list-group-item-value\">{{primaryItem.label}}</span> <span ng-if=\"nav.collapsed && !isMobile\" class=\"sr-only\">{{primaryItem.label}}</span>\n" +
494494
"</a>\n" +
495495
"<a ng-if=\"!primaryItem.href\" href=\"\" ng-click=\"itemClicked(primaryItem)\">\n" +
496-
"<span class=\"{{primaryItem.iconClass}}\"></span> <span class=\"list-group-item-value\">{{primaryItem.label}}</span>\n" +
496+
"<span title=\"{{primaryItem.label}}\" class=\"{{primaryItem.iconClass}}\" aria-hidden=\"true\"></span> <span class=\"list-group-item-value\">{{primaryItem.label}}</span> <span ng-if=\"nav.collapsed && !isMobile\" class=\"sr-only\">{{primaryItem.label}}</span>\n" +
497497
"</a>\n" +
498498
"\n" +
499499
"<div ng-if=\"primaryItem.secondaryNavSections.length\" class=\"secondary-nav-item-pf\" ng-class=\"{\n" +

0 commit comments

Comments
 (0)