# Objective
use pickers for all kinds of lsp definitions

## Solution

add handlers for `GoToDeclaration` and `GoToTypeDefinition` in
`lsp_locations.rs`

## Testing

- Did you test these changes? If so, how?
- Are there any parts that need more testing?
- How can other people (reviewers) test your changes? Is there anything
specific they need to know?
- If relevant, what platforms did you test these changes on, and are
there any important ones you can't test?

## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

## Showcase

this rust code can be used to test for multiple type definitions on `a`
```rs
macro_rules! foo {
    ($e:ident) => {
        let $e: Vec<()>;
        let $e: String;
    };
}
fn main() {
    foo!(a);
}
```
<img width="735" height="288" alt="image"
src="https://github.com/user-attachments/assets/578e6dc2-b201-4194-bd8b-e8d17b4d3d50"
/>

---

Release Notes:

- Respect `"lsp_results_location": "picker"` for go to declaration and
type definition