Skip to content

Commit 166e09e

Browse files
Restore code that removed separators on filter (microsoft#181289)
A version of this code was removed when I added quick pick item separator buttons... causing microsoft#180037 This restores that code. Fixes microsoft#180037
1 parent df9f155 commit 166e09e

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/vs/platform/quickinput/browser/quickInputList.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -804,6 +804,13 @@ export class QuickInputList {
804804
element.hidden = element.item ? !element.item.alwaysShow : true;
805805
}
806806

807+
// Ensure separators are filtered out first before deciding if we need to bring them back
808+
if (element.item) {
809+
element.separator = undefined;
810+
} else if (element.separator) {
811+
element.hidden = true;
812+
}
813+
807814
// we can show the separator unless the list gets sorted by match
808815
if (!this.sortByLabel) {
809816
const previous = element.index && this.inputElements[element.index - 1];

0 commit comments

Comments
 (0)