Skip to content
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

center swiper-isearch result in buffer #2146

Closed
CDitzel opened this issue Jul 18, 2019 · 10 comments
Closed

center swiper-isearch result in buffer #2146

CDitzel opened this issue Jul 18, 2019 · 10 comments

Comments

@CDitzel
Copy link

@CDitzel CDitzel commented Jul 18, 2019

After successfully using Swiper to find something in the current buffer, i hit Return to confirm the position change. How can I ensure, that the line with the search result in the actual buffer becomes the center line of the current buffer? That would be both helpful and more intuitive

@manuel-uberti
Copy link
Contributor

@manuel-uberti manuel-uberti commented Jul 18, 2019

Does setting swiper-action-recenter to t work for you?

@CDitzel
Copy link
Author

@CDitzel CDitzel commented Jul 18, 2019

unfortunately it does not. My config always has been

(use-package swiper
  :bind (("C-s" . swiper-isearch)
         :map swiper-map
         ("M-%" . swiper-query-replace))
  :custom
  (swiper-action-recenter t))

@manuel-uberti
Copy link
Contributor

@manuel-uberti manuel-uberti commented Jul 18, 2019

swiper-action-recenter works for regular swiper, not for swiper-isearch.

@CDitzel
Copy link
Author

@CDitzel CDitzel commented Jul 18, 2019

mh thanks, is there some work around?

@manuel-uberti
Copy link
Contributor

@manuel-uberti manuel-uberti commented Jul 18, 2019

Not that I'm aware of. I'd suggest renaming the issue title mentioning you're using swiper-isearch, just to make it clearer for @abo-abo at a quick glance.

@CDitzel CDitzel changed the title center swiper search result in buffer center swiper-isearch result in buffer Jul 18, 2019
@abo-abo abo-abo closed this in 54d1b19 Jul 18, 2019
@abo-abo
Copy link
Owner

@abo-abo abo-abo commented Jul 18, 2019

Should work now. Thanks.

@CDitzel
Copy link
Author

@CDitzel CDitzel commented Jul 18, 2019

it does not work. Pulled the latest commit and the reported behaviour prevails

@abo-abo
Copy link
Owner

@abo-abo abo-abo commented Jul 18, 2019

@CDitzel Please make sure you reload/recompile. I reproduce the correct behavior with make plain.

@CDitzel
Copy link
Author

@CDitzel CDitzel commented Jul 18, 2019

how do I do that?

I did straight-pull-package swiper and straight-rebuild-all but it does not work

@abo-abo
Copy link
Owner

@abo-abo abo-abo commented Jul 18, 2019

@CDitzel

git clone https://github.com/abo-abo/swiper
cd swiper
make plain

astoff added a commit to astoff/swiper that referenced this issue Jan 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants