-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
"bad scroll event" occurring in Command Palette in Theia if the application does not include the "@theia/ai-chat-ui" module #15042
Labels
Comments
Observations:
|
sdirix
added a commit
to eclipsesource/theia
that referenced
this issue
Feb 26, 2025
Makes sure the global Monaco BaseLayerHoverDelegate is properly initialized with Monaco's HoverService before opening a quick input. This fixes rendering issues caused by uncaught errors when opening a quick input, for example the command palette, before any Monaco editor was initialized. This scenario was easily reproducible in minimal Theia applications, not leveraging all offered Theia extensions. fixes eclipse-theia#15042
2 tasks
sdirix
added a commit
to eclipsesource/theia
that referenced
this issue
Feb 26, 2025
Makes sure the global Monaco BaseLayerHoverDelegate is properly initialized with Monaco's HoverService before opening a quick input. This fixes rendering issues caused by uncaught errors when opening a quick input, for example the command palette, before any Monaco editor was initialized. This scenario was easily reproducible in minimal Theia applications, not leveraging all offered Theia extensions. fixes eclipse-theia#15042
sdirix
added a commit
to eclipsesource/theia
that referenced
this issue
Feb 26, 2025
Makes sure the global Monaco BaseLayerHoverDelegate is properly initialized with Monaco's HoverService. This fixes rendering issues caused by uncaught errors when opening a quick input, for example the command palette, before any Monaco editor was initialized. This scenario was easily reproducible in minimal Theia applications, not leveraging all offered Theia extensions. fixes eclipse-theia#15042
sdirix
added a commit
that referenced
this issue
Feb 26, 2025
Makes sure the global Monaco BaseLayerHoverDelegate is properly initialized with Monaco's HoverService. This fixes rendering issues caused by uncaught errors when opening a quick input, for example the command palette, before any Monaco editor was initialized. This scenario was easily reproducible in minimal Theia applications, not leveraging all offered Theia extensions. fixes #15042
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bug Description:
When starting to develop a fresh extension using the yeoman code generator, the default electron-app and browser-app folders each contain a
package.json
listing a minimum Theia environment as dependencies.But when I build an run these applications, and in the fresh, empty Theia application, I invoke the Command Palette (using CMD+SHIFT+P), the Command Palette is not cleanly formatted, but looks distorted.
Additionally, on the console, this error is logged:
I have started to debug this by including the full set of dependencies from Theia IDE and one by one stripping them.
For some reason, the problem seems to be that the "@theia/ai-chat-ui" dependency is missing, because, when I add it to the minimal application, the Command Palette works as expected; but when I remove it again, the Command Palette is distorted again.
Maybe there is an implicit dependency somewhere that causes this? (Or it is a transitive dependency of the ai-chat-ui module ...?).
In any case, the yeoman generator should be fixed to include the necessary dependency (or better yet, if it is really the missing ai-chat-ui module that causes this, I would expect the minimal Theia to even work fine without any AI module installed...).
Steps to Reproduce:
yo theia-extension
yarn && yarn build:electron && yarn start:electron
Additional Information
The text was updated successfully, but these errors were encountered: