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

input not at force editor #7902

Closed
502647092 opened this issue May 27, 2020 · 2 comments · Fixed by #7903
Closed

input not at force editor #7902

502647092 opened this issue May 27, 2020 · 2 comments · Fixed by #7903
Assignees
Labels
bug bugs found in the application editor issues related to the editor

Comments

@502647092
Copy link
Contributor

Bug Description:

github

Steps to Reproduce:

I don't know how to reproduce it
but It happened several times

Additional Information

  • Operating System: Windows10 Chrome 83.0.4103.61
  • Theia Version: 1.2.0-next.f7cecf20
@akosyakov akosyakov self-assigned this May 27, 2020
@akosyakov
Copy link
Member

@502647092 Could you reproduce it with 1bc44b3? Your version does not seem to contain it.

@akosyakov akosyakov added bug bugs found in the application editor issues related to the editor labels May 27, 2020
@akosyakov
Copy link
Member

From reading the code I think i understand the issue now:

  • We apply keystrokes either to focused monaco editor or last focused tracked by us. We have to do it since we don’t track inline (debug console) and embedded editors (editor in peek reference), but Monaco does. Plus it is the same what VS Code do.
  • When we ask Monaco to focus something it expects that the editor is already revealed and forcefully override hasFocus flag.
  • So in the situation when the widget is not revealed we apply keystrokes to wrong Monaco editor.

I can reproduce it by triggering Go to symbol… and changing the editor. First the quick pick does not get closed on an editor change and when i close it, it tries to focus the old editor.

akosyakov added a commit that referenced this issue May 27, 2020
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
akosyakov added a commit that referenced this issue May 27, 2020
Signed-off-by: Anton Kosyakov <anton.kosyakov@typefox.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs found in the application editor issues related to the editor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants