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

Command Palette in 1.1.0.RC10 - wrong position of suggestion list? #1094

Closed
planger opened this issue Aug 21, 2023 Discussed in #1093 · 3 comments · Fixed by eclipse-glsp/glsp-client#277
Closed

Command Palette in 1.1.0.RC10 - wrong position of suggestion list? #1094

planger opened this issue Aug 21, 2023 Discussed in #1093 · 3 comments · Fixed by eclipse-glsp/glsp-client#277
Assignees
Labels

Comments

@planger
Copy link
Member

planger commented Aug 21, 2023

Discussed in #1093

Originally posted by rsoika August 20, 2023
Hi,
something seems to have changed with the Command Palette from RC04 to RC10 with the

The command-palette-suggestions are now no longer placed relative to the command-palette-input. It seems the element is now positioned relative to the man window. If I change the left menue-bar than the command-palette-suggestions position also changed.

See:

Peek 2023-08-20 15-04

I am not the expert in CSS but I think the problem is the attribute position: absolute; assigned to the element autocomplete command-palette-suggestions If I remove this attribute or change it to fixed it may propabyl work again correctly. But I can not really verifiy this as the component is constructed by some javascript dynamically.

@planger planger added the client label Aug 21, 2023
@tortmayr
Copy link
Contributor

This seems to be broken in sprotty as well. I think we pulled in this bug when upgrading to the latest sprotty version.

@tortmayr
Copy link
Contributor

@planger I provided a fix in sprotty for this: eclipse-sprotty/sprotty#372.

@planger
Copy link
Member Author

planger commented Aug 22, 2023

Great, thank you @tortmayr! Already briefly tested and approved :-)

tortmayr added a commit to eclipse-glsp/glsp-client that referenced this issue Aug 24, 2023
Update to sprotty 0.14.0 and adapt code to conform to latest changes/API breaks.
With 0.14.0 the `ToolManager API` was deprecated and it will be removed in future versions.
The intention is that this API moves to GLSP as it is only used there => refactor `ToolManager API`

Fixes eclipse-glsp/glsp#1096
Fixes eclipse-glsp/glsp#1094
tortmayr added a commit to eclipse-glsp/glsp-client that referenced this issue Aug 25, 2023
Update to sprotty 0.14.0 and adapt code to conform to latest changes/API breaks.
With 0.14.0 the `ToolManager API` was deprecated and it will be removed in future versions.
The intention is that this API moves to GLSP as it is only used there => refactor `ToolManager API`

GLSP-960: Make glspClient retrieval async

Fixes eclipse-glsp/glsp#1096
Fixes eclipse-glsp/glsp#1094
tortmayr added a commit to eclipse-glsp/glsp-client that referenced this issue Aug 25, 2023
Update to sprotty 0.14.0 and adapt code to conform to latest changes/API breaks.
With 0.14.0 the `ToolManager API` was deprecated and it will be removed in future versions.
The intention is that this API moves to GLSP as it is only used there => refactor `ToolManager API`

GLSP-960: Make glspClient retrieval async

Fixes eclipse-glsp/glsp#1096
Fixes eclipse-glsp/glsp#1094
tortmayr added a commit to eclipse-glsp/glsp-client that referenced this issue Aug 25, 2023
Update to sprotty 0.14.0 and adapt code to conform to latest changes/API breaks.
With 0.14.0 the `ToolManager API` was deprecated and it will be removed in future versions.
The intention is that this API moves to GLSP as it is only used there => refactor `ToolManager API`

GLSP-960: Make glspClient retrieval async

Fixes eclipse-glsp/glsp#1096
Fixes eclipse-glsp/glsp#1094
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants