-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
helm-swoop does not work as expected anymore #13196
Comments
Confirmed. Windows 1903#### System Info :computer: - OS: windows-nt - Emacs: 26.3 - Spacemacs: 0.300.0 - Spacemacs branch: develop (rev. d7e9a803f) - Graphic display: t - Distribution: spacemacs - Editing style: vim - Completion: helm - Layers: ```elisp (autohotkey auto-completion colors command-log emacs-lisp git haskell helm imenu-list latex markdown multiple-cursors (org :variables org-agenda-files '("c:/Users/username/org/notes.org")) python (shell :variables shell-default-shell 'eshell) spell-checking syntax-checking treemacs version-control) ``` - System configuration features: XPM JPEG TIFF GIF PNG RSVG SOUND NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS THREADS LCMS2 |
It seems to be caused by Then helm-swoop |
Correction it might not have been caused by: There is a newer version available when I just checked: |
This is an upstream issue, I've tried with a completely vanilla emacs, helm-core / helm-swoop installed from melpa with the same results. As a sidenote, it isn't immediately clear to me how can I temporarily revert to that version with spacemacs, can you please elaborate? |
When updating packages, the outdated packages from the five previous times are stored in: The current version is stored in: Move the current version some where else and copy the previous version from a rollback dir and paste it where the current version was. Unfortunately it won't work if one just has cloned a fresh copy of Spacemacs, because it downloads the latest versions from melpa. |
Thanks for testing it without Spacemacs. |
I found some notes about a possible way to load a package at a specific commit.
source: #12143 (comment) An article with a similar suggestion: |
This seems to be the helm-swoop repository: Feel free to report it, I'm sure they also would like to get it fixed. |
I've already tracked down the commit from helm that seems to break helm-swoop, I'll be reporting shortly. Should this issue be closed? |
was able to rollback to a working config. In case it helps anyone in the meantime.
|
@brunosmmm Leave it open until it's fixed. |
Reported. Just for reference, this behavior shows up from upstream helm commit emacs-helm/helm@c09329c helm-swoop does not work as expected emacsorphanage/helm-swoop#175
|
Confirmed! I was just about to report the same bug. |
The I just updated the packages and the Now when I search for |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this issue is still valid! |
Description
helm-swoop does not work as expected
Reproduction guide 🪲
Observed behaviour: 👀 💔
helm-swoop buffer displays entire buffer that is being searched with occurences highlighted, not just lines with occurrences, making it impossible to jump between occurrences
Expected behaviour: ❤️ 😄
helm-swoop buffer displays only lines with occurrences of the search string, able to jump between buffer locations by navigating up/down helm-swoop buffer
System Info 💻
The text was updated successfully, but these errors were encountered: