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

lsp: Respect the SHOW property and improve expansion state for nodes without children #224

Merged
merged 2 commits into from
Jan 16, 2025

Conversation

NiklasRentzCAU
Copy link
Member

@NiklasRentzCAU NiklasRentzCAU commented Jan 16, 2025

Allow to expand nodes that have no children so that the expanded rendering is shown.
This is now consistent with the Eclipse implementation - any expanded node gets the "populated" property regardless of if it has children. This fixes the "Details" button in the EObjectFallbackSynthesis to properly show the details again.

Also, respect the SHOW property to allow to hide graph elements.

rendering is shown.

This is now consistent with the Eclipse implementation - any expanded
node gets the "populated" property regardless of if it has children.
This fixes the "Details" button in the EObjectFallbackSynthesis to
properly show the details again.
@NiklasRentzCAU NiklasRentzCAU added bug Something isn't working LSP Affect the klighd language server. labels Jan 16, 2025
@NiklasRentzCAU NiklasRentzCAU self-assigned this Jan 16, 2025
@NiklasRentzCAU NiklasRentzCAU changed the title lsp: allow to expand nodes that have no children so that the expanded rendering is shown. lsp: Respect the SHOW property and improve expansion state for nodes without children Jan 16, 2025
@NiklasRentzCAU NiklasRentzCAU merged commit 9ee820e into master Jan 16, 2025
3 checks passed
@NiklasRentzCAU NiklasRentzCAU deleted the nre/expandNodeWithoutChildren branch January 16, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working LSP Affect the klighd language server.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant