diff options
| author | Annabel Dunstone Gray <annabel.dunstone@gmail.com> | 2017-08-30 17:34:48 +0000 |
|---|---|---|
| committer | Annabel Dunstone Gray <annabel.dunstone@gmail.com> | 2017-08-30 17:34:48 +0000 |
| commit | b190c6abe78f0fd75b6728f939a87ae73e41b07f (patch) | |
| tree | a20e2988bab3d05e9eabeac387451292eae1a98c /app/assets/javascripts | |
| parent | e009fa5746617468fef29d7e9564a74ebaa5e5ce (diff) | |
| parent | 0ab0e7b3bb3d83730e3bd5cca8258d2952ef5976 (diff) | |
| download | gitlab-ce-b190c6abe78f0fd75b6728f939a87ae73e41b07f.tar.gz | |
Merge branch 'remove-sidebar-title' into 'master'
Removed tooltips from new sidebar
Closes #35658
See merge request !13489
Diffstat (limited to 'app/assets/javascripts')
| -rw-r--r-- | app/assets/javascripts/new_sidebar.js | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/app/assets/javascripts/new_sidebar.js b/app/assets/javascripts/new_sidebar.js index b18d12b48b5..05e3f33f5ed 100644 --- a/app/assets/javascripts/new_sidebar.js +++ b/app/assets/javascripts/new_sidebar.js @@ -15,6 +15,7 @@ export default class NewNavSidebar { this.$openSidebar = $('.toggle-mobile-nav'); this.$closeSidebar = $('.close-nav-button'); this.$sidebarToggle = $('.js-toggle-sidebar'); + this.$topLevelLinks = $('.sidebar-top-level-items > li > a'); } bindEvents() { @@ -50,6 +51,10 @@ export default class NewNavSidebar { this.$page.toggleClass('page-with-icon-sidebar', breakpoint === 'sm' ? true : collapsed); } NewNavSidebar.setCollapsedCookie(collapsed); + + this.$topLevelLinks.attr('title', function updateTopLevelTitle() { + return collapsed ? this.getAttribute('aria-label') : ''; + }); } render() { |