-
Notifications
You must be signed in to change notification settings - Fork 15
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
Comments
Hmm, not sure, I’ll have to install selectrum to test this. Thanks for the report!
… On Dec 23, 2021, at 1:21 PM, Brian Claywell ***@***.***> wrote:
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!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
|
I'm seeing the same thing with vertico. |
Ended up here because of vertico too.
Ah, yes, selectrum and vertico causes window resizing, which triggers The problem gets even worse with consult that'll preview the buffer in |
I’ve been swamped in day-job work but sounds like I need to take another run at dimmer bugs and compatibility. Thanks for your patience!On May 5, 2024, at 2:43 PM, Thomas Fini Hansen ***@***.***> wrote:
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.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>
|
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. |
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.
I'm having trouble with the active window being dimmed when the selectrum minibuffer opens, despite
window-minibuffer-p
being indimmer-prevent-dimming-predicates
. I've tried various things. There's a buffer-local variable calledselectrum-is-active
that I tried writing a simple predicate for: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!
The text was updated successfully, but these errors were encountered: