Skip to content

Commit 7a8b6a8

Browse files
authored
Merge pull request microsoft#200591 from jeanp413/fix-200590
Fixes breadcrumbs widget does not get resized properly
2 parents e763a3d + eb7a51c commit 7a8b6a8

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/vs/base/browser/ui/breadcrumbs/breadcrumbsWidget.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,7 @@ export class BreadcrumbsWidget {
5757
private _focusedItemIdx: number = -1;
5858
private _selectedItemIdx: number = -1;
5959

60+
private _pendingDimLayout: IDisposable | undefined;
6061
private _pendingLayout: IDisposable | undefined;
6162
private _dimension: dom.Dimension | undefined;
6263

@@ -100,6 +101,7 @@ export class BreadcrumbsWidget {
100101
dispose(): void {
101102
this._disposables.dispose();
102103
this._pendingLayout?.dispose();
104+
this._pendingDimLayout?.dispose();
103105
this._onDidSelectItem.dispose();
104106
this._onDidFocusItem.dispose();
105107
this._onDidChangeFocus.dispose();
@@ -112,11 +114,12 @@ export class BreadcrumbsWidget {
112114
if (dim && dom.Dimension.equals(dim, this._dimension)) {
113115
return;
114116
}
115-
this._pendingLayout?.dispose();
116117
if (dim) {
117118
// only measure
118-
this._pendingLayout = this._updateDimensions(dim);
119+
this._pendingDimLayout?.dispose();
120+
this._pendingDimLayout = this._updateDimensions(dim);
119121
} else {
122+
this._pendingLayout?.dispose();
120123
this._pendingLayout = this._updateScrollbar();
121124
}
122125
}

0 commit comments

Comments
 (0)