Hello Paul, On 20/12/2025, Paul D. Nelson wrote: > OK, makes sense. I'm happy to leave this aspect as it is for now. > > Another related point: when preview-keep-still-images is t and > preview-visibility-style is 'off-point, as the preview starts > regenerating, the construction icon flashes. I feel like setting > preview-keep-still-images to t should universally suppress the > construction icon. It does when preview-visibility-style is 'always, so > why not also when it is 'off-point? (If it did, then > preview-keep-still-images would retain some relevance in the 'off-point > case.)
Yes. As noted, when `preview-visibility-style` is `off-point`, `preview-keep-still-images` is ignored; so what you describe is intended behaviour. I take your point about suppressing the construction icon. That said, the behaviour follows from the earlier discussion about keeping the preview open when modified. Showing an icon immediately before generation, then briefly flashing the old preview on regeneration, risks suggesting the preview failed to update, before it is replaced by the correct one. On the other hand, I am not sure what the benefit of such behaviour would be. This behaviour would be even stranger once the preview correctly remains open during regeneration (which I haven't fixed yet). > If I recall correctly, I did get the chance to test the latest buframe > code on a multi-monitor setup earlier this week and that it continued to > misbehave, but I didn't get the chance to look closely at this. OK. I just had a chance to test on a multi-monitor setup (on macOS FWIW). The new "fix" is indeed wrong. However, after undoing the fix (i.e., using the old way of getting window coordinates), the previews with buframes work flawlessly on my setup with either monitor. So we're gonna have to dig deeper into this issue on your system. One way to proceed is if you can tell me what your OS/windowing-system are and what does `window-inside-pixel-edges` and `frame-position` return when called on a new frame in different monitors (the first return value should be the same while the latter should be different). > One thought about deprecation: I think the now-obsolete variable > preview-leave-open-previews-visible should have a custom setter that > sets preview-visibility-style to 'always and preview-keep-still-images > to t, since that would give the most faithful reproduction of previous > behavior. (Happy to elaborate or provide code if that'd help.) Agreed. I'll add that code to the next version of the patch. Best regards, -- Al _______________________________________________ bug-auctex mailing list [email protected] https://lists.gnu.org/mailman/listinfo/bug-auctex
