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

style(lsp): make the base LSP label more like others #624

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

awilkins
Copy link

Description

The base entry has an label with the LSP client name in
parentheses, which pushes the range to the right, making
it more likely that the range will leave the edge of the viewport.

Since the range is more useful information than the LSP client,
change the base label format to be more like the others.

This also improves the appearance of the tree.

e.g.

     └╴1 ACCOUNT-ENTRY-VIEW VIEW OF ACCOUNT-ENTRY (natls) [105, 3]

     becomes                                  highlighted  v---v

     └╴1 ACCOUNT-ENTRY-VIEW VIEW OF ACCOUNT-ENTRY [105, 3] natls

The base entry had an label with the LSP client name in
parentheses, which pushes the range to the right, making
it more likely that the range will leave the edge of the viewport.

Since the range is more useful information than the LSP client,
change the base label format to be more like the others.

This also improves the appearance of the tree.

e.g.

```
     └╴1 ACCOUNT-ENTRY-VIEW VIEW OF DLP-ACCOUNT-ENTRY (natls) [105, 3]

     becomes                                      highlighted  v---v

     └╴1 ACCOUNT-ENTRY-VIEW VIEW OF DLP-ACCOUNT-ENTRY [105, 3] natls
```
@awilkins awilkins changed the title Make the base LSP label more like others style: Make the base LSP label more like others Feb 13, 2025
@awilkins awilkins changed the title style: Make the base LSP label more like others style: make the base LSP label more like others Feb 13, 2025
@awilkins awilkins changed the title style: make the base LSP label more like others style(lsp): make the base LSP label more like others Feb 13, 2025
Copy link
Contributor

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Mar 16, 2025
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 this pull request may close these issues.

1 participant