-
-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature request: ability to select objects in overlays #255
Comments
It's not clear for me what you're trying to do. Please elaborate. I saw your example with "link in overlay" button. But navigating there might not be a well posed problem, since the overlay doesn't occupy a point, it's not possible to move the point there, only to some other point close to it. At that point, you can just detect where your overlays are and use plain-old |
Sorry for the vague description. In I want to be able to make them accessible with the keyboard and link-hint(which uses avy display engine) seems to be a good fit for solving this problem. I want to achieve the following: Pass a list of overlays + position in overlay and avy will allow selecting the item just like you can do that with |
This is doable, maybe in a slightly different way. Like I said, overlays don't occupy a point, so it doesn't make sense for me to splice an avy overlay in the middle of a different overlay. Here's an example:
Just pass the point positions of the overlay, get the selected position back and locate the overlay by position. |
@abo-abo https://user-images.githubusercontent.com/13259670/51136922-4ff8e600-1846-11e9-9275-c668314082e5.png |
I followed the avy--process handling and I believe that passing custom display-fn and cleanup-fn will possibly be able to handle our case. Thanks for the pointer |
Fixes abo-abo#255 - I have renamed it to avy-process to indicate that it is used from another packages.
One of the drawbacks of displaying actions in overlays similar to what
lsp-ui
is doing is that you cannot "trigger" to them using keyboard. It would be good ifavy
supports overlay as a target location. See noctuid/link-hint.el#24 for more detailed discussion.@noctuid
The text was updated successfully, but these errors were encountered: