Commit d1067acd authored by Filipa Lacerda's avatar Filipa Lacerda

Merge branch 'issuable-sidebar-collapsed-labels-icon' into 'master'

Fixed labels icon being hidden in collapsed issuable sidebar

Closes #29751

See merge request !10107
parents 1bec2fb3 93006905
...@@ -121,7 +121,7 @@ ...@@ -121,7 +121,7 @@
- selected_labels = issuable.labels - selected_labels = issuable.labels
.block.labels .block.labels
.sidebar-collapsed-icon.js-sidebar-labels-tooltip{ title: issuable_labels_tooltip(issuable.labels_array), data: { placement: "left", container: "body" } } .sidebar-collapsed-icon.js-sidebar-labels-tooltip{ title: issuable_labels_tooltip(issuable.labels_array), data: { placement: "left", container: "body" } }
= icon('tags', class: 'hidden', 'aria-hidden': 'true') = icon('tags', 'aria-hidden': 'true')
%span %span
= selected_labels.size = selected_labels.size
.title.hide-collapsed .title.hide-collapsed
......
...@@ -78,5 +78,11 @@ import '~/right_sidebar'; ...@@ -78,5 +78,11 @@ import '~/right_sidebar';
expect(todoToggleSpy.calls.count()).toEqual(1); expect(todoToggleSpy.calls.count()).toEqual(1);
}); });
it('should not hide collapsed icons', () => {
[].forEach.call(document.querySelectorAll('.sidebar-collapsed-icon'), (el) => {
expect(el.querySelector('.fa, svg').classList.contains('hidden')).toBeFalsy();
});
});
}); });
}).call(window); }).call(window);
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