-
Notifications
You must be signed in to change notification settings - Fork 30
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
New feature: bb-jump: quickly navigate directly to any bm #37
base: master
Are you sure you want to change the base?
Conversation
I am not getting this to work properly:
Isn't this use-case already solved by the functionality of bm-show and bm-show-all? |
On 2021-05-05 13:01, Jo Odland wrote:
I am not getting this to work properly:
* No auto-complete in the minibuffer just a history list to scroll up
or down.
It's using the standard emacs function 'completing-read' so how-ever you
personally have that set up for completion should work. You can see on
line 59 that a collection is being given as the second arg to completing
read and NO history argument is being provided. Maybe you need to press
TAB twice? There are many front ends for completing-read (eg. helm, ido,
ivy, icicles) so it may depend on what you have installed.
* Jumping to a bookmark in a buffer other than the current buffer
does not work.
That's a bug. It should be fixed if between lines 597-598 you add:
(pop-to-buffer (overlay-buffer (cadr decision)))
so:
(when (setq decision (assoc decision candidates))
(goto-char (overlay-start (cadr decision))))))
becomes:
(when (setq decision (assoc decision candidates))
(pop-to-buffer (overlay-buffer (cadr decision)))
(goto-char (overlay-start (cadr decision))))))
Can this use-case be solved by the functionality of bm-show?
I wanted to offer something quicker, that didn't entail creating a new
buffer, and then needing to navigate through it. The behavior is modeled
on the behavior of the emacs 'C-x r d' (M-x bookmark-jump), but with a
prettier and more informative display.
Strange / curious about the lack of auto-complete, though. Do you have
any more details to share about that?
Also, let me know if you're ok with the additional line at 597-598
Did you get a chance to look at the other PR?
…--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1 7286 0036 9E45 1595 8BC0
|
"Jump to bookmark: " | ||
(mapcar 'car candidates) nil t))) | ||
(when (setq decision (assoc decision candidates)) | ||
(goto-char (overlay-start (cadr decision)))))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I understand that the bookmark is not available, but I don't like that jumping to a bookmark is done without using bm-goto() since there are more things happening in addition to the jump.
On 2021-05-05 15:27, Jo Odland wrote:
I understand that the bookmark is not available, but I don't like that
jumping to a bookmark is done without using bm-goto() since there are
more things happening in addition to the jump.
I don't understand. If you're saying to just replace the the line (now
two lines) with function bm-goto, just do it! What's the big deal, then?
Are you asking me to do it?
…--
hkp://keys.gnupg.net
CA45 09B5 5351 7C11 A9D1 7286 0036 9E45 1595 8BC0
|
Inspired by function
bookmark-jump
from emacs-core packagebookmark.el
List of bookmarks are presented as mini-buffer completion candidates
Presentation includes enhanced fontification of each bookmark's file name and line number.