Skip to content

project_symbols: Display line numbers in symbol picker#46507

Merged
mattermill merged 2 commits intozed-industries:mainfrom
aviatesk:project-symbol-line
Jan 12, 2026
Merged

project_symbols: Display line numbers in symbol picker#46507
mattermill merged 2 commits intozed-industries:mainfrom
aviatesk:project-symbol-line

Conversation

@aviatesk
Copy link
Contributor

Some language servers return local symbols in workspace/symbol responses, and languages like Julia can have multiple overloads for the same generic function name. In these cases, symbols may appear identical in the symbol picker, making it hard to distinguish between them. Adding line numbers helps users identify the correct symbol.

In the future, we may also want to consider using containerName from the LSP response for additional context.

Before

Screenshot 2026-01-10 at 19 21 08

After

Screenshot 2026-01-10 at 19 43 19

Closes #ISSUE

Release Notes:

  • Added line numbers to the project symbols picker to help distinguish between symbols with the same name

Some language servers return local symbols in `workspace/symbol`
responses, and languages like Julia can have multiple overloads for the
same generic function name. In these cases, symbols may appear
identical in the symbol picker, making it hard to distinguish between
them. Adding line numbers helps users identify the correct symbol.

In the future, we may also want to consider using `containerName` from
the LSP response for additional context.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jan 10, 2026
@mattermill
Copy link
Contributor

@aviatesk Thanks for tackling this! Made a small change to differentiate the line number from the path. I was finding it difficult to parse the line number when scanning the list.

@mattermill mattermill enabled auto-merge (squash) January 12, 2026 17:12
@aviatesk
Copy link
Contributor Author

Made a small change to differentiate the line number from the path.

Sounds good:) Thanks for your review.

@mattermill mattermill merged commit 1148773 into zed-industries:main Jan 12, 2026
35 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants