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

selectrum compatibility #60

Open
bcclaywell opened this issue Dec 23, 2021 · 5 comments
Open

selectrum compatibility #60

bcclaywell opened this issue Dec 23, 2021 · 5 comments

Comments

@bcclaywell
Copy link

I'm having trouble with the active window being dimmed when the selectrum minibuffer opens, despite window-minibuffer-p being in dimmer-prevent-dimming-predicates. I've tried various things. There's a buffer-local variable called selectrum-is-active that I tried writing a simple predicate for:

(defun selectrum-is-active-p ()
  (and selectrum-is-active t))

That doesn't seem to do the trick, the active window still gets dimmed when the focus moves to the selectrum minibuffer. Not sure what else to try, but it would be great if selectrum and dimmer could work together!

@gonewest818
Copy link
Owner

gonewest818 commented Dec 24, 2021 via email

@Stebalien
Copy link

I'm seeing the same thing with vertico. dimmer-process-all is getting called with force by dimmer-config-change-handler (at least for me).

@xendk
Copy link

xendk commented May 5, 2024

Ended up here because of vertico too.

dimmer-process-all is getting called with force by dimmer-config-change-handler

Ah, yes, selectrum and vertico causes window resizing, which triggers window-configuration-change-hook, which dimmer hooks into.

The problem gets even worse with consult that'll preview the buffer in consult-buffer.

@gonewest818
Copy link
Owner

gonewest818 commented May 5, 2024 via email

@xendk
Copy link

xendk commented May 6, 2024

I think it could be fixable for selectrum and vertico, but either with consult is a tough nut. Basically, dimmer would have to hook into consult previewing and "keep up" with it while one browse the selection. I'm not sure if it would be worth the effort.

It kinda makes sense that the originating buffer gets dimmed when selectrum/vertico is active, and I, personally, of course, don't mind it. So I would probably prefer if the current behaviour was still an option, as a halfway solution that doesn't dim the originating buffer, but doesn't keep up when consult previews buffers/files would be more jarring.

xendk added a commit to xendk/dotemacs that referenced this issue May 8, 2024
There's already an upstream issue, but it turns out that consult
further complicates the issue.

gonewest818/dimmer.el#60 (comment)

I'll live with the current behaviour.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants