b1639e5
Add cmd-g and cmd-shift-g to jump to next / previous result
Click to expand commit body
I added the action handler on Pane so we can use these bindings when the find bar isn't focused.
Nathan Sobo
created
f90193b
Populate query and select it only if find bar isn't already deployed
Antonio Scandurra
created
83423a4
Use cmd-f to move focus back to the editor when find bar is focused
Antonio Scandurra
created
9ce1eda
Bind `cmd-e` to deploy `FindBar` without focusing it
Antonio Scandurra
created
ce52704
Propagate `Cancel` when editor is not in full mode
Click to expand commit body
This is consistent with what VS Code and Sublime Text do and allows the user to
perform only one keybinding to e.g. dismiss the find bar when the query is
(partially) selected.
Antonio Scandurra
created
c53b6b9
Populate query with text under selection when hitting `cmd-f`
Antonio Scandurra
created
2a1b1ad
Bind `enter` and `shift-enter` in `FindBar`
a284e71
Always return valid locations when refreshing anchors
Click to expand commit body
Specifically, with this commit:
- We will now refresh the anchor if it escapes the boundaries of the excerpt by
using the `Excerpt::contains` method. This was not the case before, as we were
just checking if the excerpt id and buffer id of the anchors matched the ones
stored on the excerpt.
- We fixed a bug that was causing the anchor to be outside of the excerpt when
resetting it to one of the excerpt's endpoints after we couldn't keep its
position. This would happen because we were using `anchor_at`, which resolved
the anchor to an offset first and then converted it back into an anchor with
the given bias, which is a lossy operation. We now use `Anchor::bias` to
achieve the same goal: note that this could still lead to the anchor escaping
the excerpt's boundary when the bias doesn't match the endpoint's bias, so we
take extra care to avoid that and `min`/`max` the newly-produced anchor with
the other endpoint.
Antonio Scandurra
created
2d6e348
Prevent anchors from escaping their excerpt's range when resolving them
Click to expand commit body
This could happen if an anchor was created on an excerpt with a larger range.
Then, if the excerpt was removed and added back at the same position and with
the same buffer but a smaller range, resolving the anchor could overshoot
the excerpt's boundaries.
This fixes a build failure on linux because gpui was being spuriously
included as a dependency of zed-server, due to its presence as an
optional dependency in the rpc crate.
Co-Authored-By: Nathan Sobo <nathan@zed.dev>