Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

editor.experimentalEditContextEnabled makes the Keyboard Shortcuts setting panel unusable #228104

Open
tamuratak opened this issue Sep 10, 2024 · 3 comments · May be fixed by #228588
Open

editor.experimentalEditContextEnabled makes the Keyboard Shortcuts setting panel unusable #228104

tamuratak opened this issue Sep 10, 2024 · 3 comments · May be fixed by #228588
Assignees

Comments

@tamuratak
Copy link
Contributor

Version: 1.94.0-insider
Commit: 8b7eb51
Date: 2024-09-10T05:04:21.546Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: 12.4.254.20-electron.0
OS: Darwin arm64 22.6.0

Steps to Reproduce:

  1. Set "editor.experimentalEditContextEnabled": true
  2. Open the Keyboard Shortcuts setting panel.
  3. We cannot type at the search bar of the panel.
@tamuratak tamuratak changed the title editor.experimentalEditContextEnabled makes Keyboard Shortcuts setting panes unusable editor.experimentalEditContextEnabled makes the Keyboard Shortcuts setting panel unusable Sep 10, 2024
@jrieken jrieken assigned hediet and aiday-mar and unassigned jrieken Sep 10, 2024
@aiday-mar
Copy link
Contributor

Another thing that happens is that when you open the settings editor. Then open the keyboard shortcuts and try to type in the input of the keyboard shortcuts nothing happens. The input of the settings editor is instead updated even though the keyboard shortcuts pane is focused.

@aiday-mar
Copy link
Contributor

In fact something weird happens. There is an edit context associated to the settings editor input which sends text update events. When we switch to the keyboard shortcuts editor and type int the input, despite the keyboard shortcuts input being clearly focused, the settings editor edit context keeps firing text update events. This can be seen in the following screen recording, where I record on the text update event, the currently focused element (which is the keyboard shortcuts input) and the dom element corresponding to the edit context which is currently firing the text update event (here it is the settings editor).

Screen.Recording.2024-09-13.at.16.47.50.mov

@aiday-mar
Copy link
Contributor

This seems to happens with the quick pick as well as with the search widget. These all share a common point in that they use the InputBox widget.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants