help-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Run `window-configuration-change-hook` in Lisp code?


From: James Cherti
Subject: Re: Run `window-configuration-change-hook` in Lisp code?
Date: Fri, 8 Nov 2024 10:51:20 -0500
User-agent: Mozilla Thunderbird

Thank you for confirming that you would place it in the
'bad practices' column.

Yes, the patch works; however, I closed the pull request as
it was too specific to another package I'm using, which restores
persisted text scale through a hook in
`window-configuration-change-hook`.

In the general case, however, the need to run
window-configuration-change-hook during text-scale-mode-hook
is not particularly important.

--
James Cherti
https://www.jamescherti.com/

On 2024-11-08 09:12, Stefan Monnier via Users list for the GNU Emacs text editor wrote:
The workaround to fix this issue is to call the following after the text
scale is changed (run-hooks 'window-configuration-change-hook)

From where?

The way I see it, another workaround is for those packages which depend
strongly on such single-size monospace text to add the relevant function
to `text-scale-mode-hook`.

OP wants to know whether it is good practice or bad practice to run the
`window-configuration-change-hook` hooks from a third-party package in this
situation, as a workaround to resolve the issue described above. (Check:
https://codeberg.org/joostkremers/visual-fill-column/pulls/16 ).

I'd put it in the bad practice column (especially since that hook
is documented to be run in a fairly special way which `run-hooks`
doesn't follow).

Then again, looking at that patch I can't understand why it would work.
Does it work?  If so how?  That code is run only when `visual-fill-mode`
is enabled, AFAICT, so whether it runs
`window-configuration-change-hook` won't make any difference when
text-scale is used.


         Stefan





reply via email to

[Prev in Thread] Current Thread [Next in Thread]