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

shorten the location of the resource in new property navigation feature #1032

Closed
martinlippert opened this issue Apr 29, 2023 · 4 comments
Closed
Assignees
Labels

Comments

@martinlippert
Copy link
Member

#761 introduced the ability to navigate from usages of properties using the @Value annotation to the definition of those properties in e.g. application.properties files. If there are multiple locations to navigate to, there is a popup showing up, letting the the user choose to which location to navigate to.

The popup shows Open Declaration - <full URI of the resource>. At the moment the absolute path of the resource is shown, which results in very long lines and unnecessary information to be shown, since the resource is located in the project or the workspace. We should shorten this line to show only the information that is necessary, e.g. only the path inside of the project or the path from the projects root onwards.

If this is something that needs to be fixed in LSP4E, we should transfer this to LSP4E and fix it there.

@BoykoAlex
Copy link
Contributor

Yes, i saw that popup... Sounds good to me - will look into this.

@BoykoAlex
Copy link
Contributor

BoykoAlex commented May 1, 2023

@martinlippert How about Open Declaration - project: (project name) - (project relative path)?

@BoykoAlex
Copy link
Contributor

LSP4E PR: eclipse-lsp4e/lsp4e#623

@BoykoAlex BoykoAlex modified the milestones: Backlog, 4.18.1.RELEASE May 2, 2023
@BoykoAlex
Copy link
Contributor

Fixed in LSP4E

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

No branches or pull requests

2 participants