kotovalexarian-likes-gitlab/gitlab-org--gitlab-foss
1
0
Fork 0
Code Releases Activity

Merge branch 'fly-out-fix-hiding' into 'master'

Browse source 
Removed display styles when hiding the fly out navigation

Closes #36224

See merge request !13431
This commit is contained in:
Filipa Lacerda 2017-08-10 08:16:55 +00:00
commit 7e82da8590
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
app/assets/javascripts/fly_out_nav.js
View file

@ -49,7 +49,8 @@ export const hideSubLevelItems = (el) => {
el.classList.remove('is-showing-fly-out');
el.classList.remove('is-over');
subItems.style.display = 'none';
subItems.style.display = '';
subItems.style.transform = '';
subItems.classList.remove('is-above');
};

2
spec/javascripts/fly_out_nav_spec.js
View file

@ -59,7 +59,7 @@ describe('Fly out sidebar navigation', () => {
expect(
el.querySelector('.sidebar-sub-level-items').style.display,
).toBe('none');
).toBe('');
});
it('does not hude subitems on mobile', () => {