maxmustermann/vscode
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Check the active element also when 'focusin' or 'focusout' bubble.

Browse source 
This change was initially done only in the editor for microsoft/monaco-editor#2339, but after a discussion we have agreed to bring it to all users of `FocusTracker`.

Co-authored-by: Johannes Rieken <johannes.rieken@gmail.com>
This commit is contained in:
Alex Dima 2021-11-22 14:59:46 +01:00
parent 7be48d527a
commit 9bd899faba
No known key found for this signature in database
GPG key ID: 39563C1504FDD0C9
2 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/vs/base/browser/dom.ts
View file

@ -978,6 +978,8 @@ class FocusTracker extends Disposable implements IFocusTracker {
this._register(addDisposableListener(element, EventType.FOCUS, onFocus, true));
this._register(addDisposableListener(element, EventType.BLUR, onBlur, true));
this._register(addDisposableListener(element, EventType.FOCUS_IN, () => this._refreshStateHandler()));
this._register(addDisposableListener(element, EventType.FOCUS_OUT, () => this._refreshStateHandler()));
}
refreshState() {

6
src/vs/editor/browser/widget/codeEditorWidget.ts
View file

@ -2007,12 +2007,6 @@ class CodeEditorWidgetFocusTracker extends Disposable {
this._hasFocus = false;
this._onChange.fire(undefined);
}));
this._register(dom.addDisposableListener(domElement, 'focusin', () => {
this._domFocusTracker.refreshState();
}));
this._register(dom.addDisposableListener(domElement, 'focusout', () => {
this._domFocusTracker.refreshState();
}));
}
public hasFocus(): boolean {