Add missing class to fix view items

For microsoft/vssaas-planning#2286
This commit is contained in:
Rob Lourens 2021-01-27 12:42:39 -08:00
parent 4378b7f020
commit 4febf1e6c3

View file

@ -192,7 +192,9 @@ export class MenuEntryActionViewItem extends ActionViewItem {
if (icon.dark) {
label.style.setProperty('--menu-entry-icon-dark', asCSSUrl(icon.dark));
}
label.classList.add('icon');
this._itemClassDispose.value = toDisposable(() => {
label.classList.remove('icon');
label.style.removeProperty('--menu-entry-icon-light');
label.style.removeProperty('--menu-entry-icon-dark');
});
@ -218,6 +220,7 @@ export class SubmenuEntryActionViewItem extends DropdownMenuActionViewItem {
container.classList.add('menu-entry');
const { icon } = (<SubmenuItemAction>this._action).item;
if (icon && !ThemeIcon.isThemeIcon(icon)) {
this.element.classList.add('icon');
if (icon.light) {
this.element.style.setProperty('--menu-entry-icon-light', asCSSUrl(icon.light));
}