helix icon indicating copy to clipboard operation
helix copied to clipboard

Published 20 hours ago • verified publisher icon helix-editor

Dedupe goto reference results

Open abseee opened this issue 3 years ago • 1 comments

We could dedupe more simply with just locations.sort(); locations.dedupe() but presumably language servers order results by relevance. Two lingering questions:

  1. How to test this
  2. This belongs in helix-lsp rather than helix-term, but I couldn't figure a way to attach a callback to the Future directly

Fixes #3351

Screenshots

Before: ksnip_20221013-053509 After: ksnip_20221013-092628

abseee avatar Oct 13 '22 14:10 abseee

Duplicate of https://github.com/helix-editor/helix/pull/4097

archseer avatar Oct 13 '22 14:10 archseer

I think we can close this now since RA merged an upstream fix for this last year: https://github.com/rust-lang/rust-analyzer/pull/13571

pascalkuthe avatar Jul 26 '23 13:07 pascalkuthe