bug#70525: [PATCH] Make auto-reveal customizations easier to extend

2024-04-22 Thread Paul Nelson
External packages (e.g., a couple that I've written) may wish to provide navigation commands that automatically reveal previews and folds, but auto-reveal behavior is currently controlled by defcustoms, such as preview-auto-reveal, that are not easily extended: (defcustom preview-auto-reveal '(e

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Arash Esbati
Hi Keita, Ikumi Keita writes: > To Arash: > It seems that the reason you saw the preview image above the TeX code is > that your inline math is at the beginning of line; in that situation, my > example indeed behaves as you described, too. Ah, Ok, `bolp' ;-) This is what I see for inline math

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Paul Nelson
> Thanks for clarification. If I understand correctly, "use the image from > the old overlay in place of the construction symbol" means that the user > can't tell whether a particular preview image is to be updated > afterwards or has already been updated, by its appearance only, during > the regen

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Ikumi Keita
Hi Paul, > Paul Nelson writes: > I think there is no such debris, and have attempted to check this, but > would be happy to learn that I missed something. The EXCEPTION > argument is used only in preview-place-preview, where it is the > overlay being generated -- not something we want to cle

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Paul Nelson
Hi Ikumi, > >> - When you enter a preview, it remains visible. The tex code appears > >> below the preview. > > The TeX code appears _to right of_ the preview for inline math > expression for me. Is this a misbehavior? Or does your above sentence > miss this case? You're right, for inlined math,

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Arash Esbati
Tassilo Horn writes: > FWIW, if we go ELPA-releases only and drop standalone tarball releases, > I would suggest we just use > > ;; Package-Requires: ... > > in our auctex.el in order to specify the required emacs version, > "cl-lib", and even "compat" and remove all legacy code for emacs > ver

bug#36114: Can't run preview latex after deleting prv files

2024-04-22 Thread Arash Esbati
Berthold Lorke writes: > I am not 100% sure what exactly my specific intention was, but I am > pretty sure that my point in the end was just that if I were to, > intentionally or accidentally, delete those files, it seems to have > "bricked" preview latex for that document. As in, I was not able

bug#66909: Frames

2024-04-22 Thread Arash Esbati
Arash Esbati writes: > Konrad Podczeck writes: > >> I have managed to have pdf-tools as a previewer in its own >> frame. However, the questions whether I wish to have an optional >> argument upon choosing a citation also happens in that frame. How can >> I proceed? (I am on macos sonoma.) > > (I

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Arash Esbati
Hi Keita, Ikumi Keita writes: > I don't mind. It doesn't matter to use emacs built-in library to write > code simply. Thanks for your response. > I'd like to ask Paul some questions: > >> Paul Nelson writes: > >>> With the new user option disabled, behavior is instead: >>> >>> - When you ent

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Tassilo Horn
Ikumi Keita writes: >> Your patch uses `cl-find-if' which is a function. Until now, we used >> only macros from the cl-lib inside AUCTeX in order to have no other >> runtime dependencies. In general, I have no problem if we lift this >> limitation and use cl-lib functions as well -- the library

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Arash Esbati
Paul Nelson writes: > Hi Arash, many thanks for taking a look. Regarding cl-find-if, I have > no strong feelings, but have attached the modified patch without it in > case that simplifies matters. Thanks Paul for your swift response. I installed that change, along with a NEWS entry. Therefore

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Ikumi Keita
Hi Arash and Paul, > Arash Esbati writes: > I don't have a suggestion for a better name, but another comment. Your > patch uses `cl-find-if' which is a function. Until now, we used only > macros from the cl-lib inside AUCTeX in order to have no other runtime > dependencies. In general, I h

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Paul Nelson
Hi Arash, many thanks for taking a look. Regarding cl-find-if, I have no strong feelings, but have attached the modified patch without it in case that simplifies matters. Thanks, best, Paul 0001-Allow-opened-previews-to-remain-visible.patch Description: Binary data ___

bug#70455: [PATCH] Add new user option `preview-leave-open-previews-visible'

2024-04-22 Thread Arash Esbati
Hi Paul, Paul Nelson writes: > With the new user option disabled (default), behavior is exactly as before: > > - When you enter a preview, it is temporarily replaced by a > placeholder icon together with the tex code. > - If you exit the preview without editing, the preview reappears and > the t