Commit c8add419 authored by Filipa Lacerda's avatar Filipa Lacerda Committed by Jose Ivan Vargas

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

Removed display styles when hiding the fly out navigation

Closes #36224

See merge request !13431
parent f6821c2d
...@@ -49,7 +49,8 @@ export const hideSubLevelItems = (el) => { ...@@ -49,7 +49,8 @@ export const hideSubLevelItems = (el) => {
el.classList.remove('is-showing-fly-out'); el.classList.remove('is-showing-fly-out');
el.classList.remove('is-over'); el.classList.remove('is-over');
subItems.style.display = 'none'; subItems.style.display = '';
subItems.style.transform = '';
subItems.classList.remove('is-above'); subItems.classList.remove('is-above');
}; };
......
...@@ -59,7 +59,7 @@ describe('Fly out sidebar navigation', () => { ...@@ -59,7 +59,7 @@ describe('Fly out sidebar navigation', () => {
expect( expect(
el.querySelector('.sidebar-sub-level-items').style.display, el.querySelector('.sidebar-sub-level-items').style.display,
).toBe('none'); ).toBe('');
}); });
it('does not hude subitems on mobile', () => { it('does not hude subitems on mobile', () => {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment