Skip to content

Return applicable ranges for hover requests #1488

@DanielRosenwasser

Description

@DanielRosenwasser

Noted from #1476, quick info/hovers currently lack a range field.

The best I can tell, the range is not necessarily an "applicable" range to avoid making the same request over and over - but it is used at least for visual indications on what the hover corresponds to.

I think without that, editors just guess at the current word.

An implementation here should be inspired by the start and end we set in QuickInfoResponseBody from the old codebase.

Metadata

Metadata

Assignees

Labels

Domain: EditorRelated to the LSP server, editor experience

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions