bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Paul Nelson
Yes, good for me. Thanks > ___ bug-auctex mailing list bug-auctex@gnu.org https://lists.gnu.org/mailman/listinfo/bug-auctex

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Arash Esbati
Paul Nelson writes: >> Sorry about that, I was sloppy in splitting up a commit. The fix is: > > In more detail, the "fix" is contained in the patch I submitted later > at #70442, and the issue arose when I split some commit into this > patch and that one. One way to proceed would be to squash

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Paul Nelson
> Sorry about that, I was sloppy in splitting up a commit. The fix is: In more detail, the "fix" is contained in the patch I submitted later at #70442, and the issue arose when I split some commit into this patch and that one. One way to proceed would be to squash those two patches, if the

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Paul Nelson
Sorry about that, I was sloppy in splitting up a commit. The fix is: (setq close-data (nconc ovl close-data)) -> (setq close-data (nconc ovl close-data))) If it's easier for you, I'm happy to resend the patch. ___ bug-auctex mailing list

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Arash Esbati
Arash Esbati writes: > Paul Nelson writes: > >> See attached, and let me know if I missed anything else. > > Thanks for your swift response. I installed the change (e4def30851), > and closing. Bad news, AUCTeX doesn't build after this commit. The error is: In toplevel form: preview.el:

bug#70443: Keep track of when previews are being generated

2024-04-17 Thread Paul Nelson
This patch makes the following changes, having no effect on behavior: - The variable preview--region-begin, introduced in the fix to Bug#62445, is replaced with preview-current-region, which is a cons cell (begin . end) keeping track of the complete region. - That variable is set to nil in

bug#70379: [PATCH] Make preview section commands preserve mark

2024-04-17 Thread Arash Esbati
Paul Nelson writes: > preview-section and preview-clearout-section activate the region, but > I don't think they should, hence this patch. Thanks, installed with commit 18a3dc7b, and closing. Best, Arash ___ bug-auctex mailing list

bug#70378: fixed patch

2024-04-17 Thread Arash Esbati
Paul Nelson writes: > I'm checking the other patches I sent to see if I messed any of them > up, and noticed that I did with this one. Attaching the fixed patch Thanks for checking Paul. I installed the last version with commit 2fe75e67, therefore closing this report. Best, Arash

bug#70442: [PATCH] Open new previews when they cover point

2024-04-17 Thread Paul Nelson
Steps to reproduce: - Create test.tex with a valid equation environment. - Put the point inside the equation environment. - preview-buffer When the preview generates, the point is ejected from the equation environment due to the new overlay. With this patch, the preview instead temporarily

bug#70370: [PATCH] Add section folding commands

2024-04-17 Thread Arash Esbati
Paul Nelson writes: > This patch adds section folding commands, so as to bring the interface > for folding more in line with that for previewing. Thanks, installed with commit f61de0abe, and closing. Best, Arash ___ bug-auctex mailing list

bug#70378: fixed patch

2024-04-17 Thread Paul Nelson
I'm checking the other patches I sent to see if I messed any of them up, and noticed that I did with this one. Attaching the fixed patch 0001-Make-locating-previews.-message-configurable.patch Description: Binary data ___ bug-auctex mailing list

bug#70369: [PATCH] Fix folding glitch with tmm disabled

2024-04-17 Thread Arash Esbati
Arash Esbati writes: > Again, can't apply the patch. Can you please try to fix as well? Scratch that, my bad, sorry. Installed with commit 5abc3d7f68. Thanks for your contribution. Closing. Best, Arash ___ bug-auctex mailing list

bug#70369: [PATCH] Fix folding glitch with tmm disabled

2024-04-17 Thread Paul Nelson
Whew, thanks. (I was about to respond with "Hmm, with this one, the patch works for me, and when I attempt to regenerate the patch using the latest fetch from master, the resulting patch is identical to the one that I had sent." Glad it worked out!) On Wed, Apr 17, 2024 at 10:35 PM Arash Esbati

bug#70369: [PATCH] Fix folding glitch with tmm disabled

2024-04-17 Thread Arash Esbati
Paul Nelson writes: > Steps to reproduce: > - Disable transient-mark-mode. > - Create test.tex containing "\emph{foo} \emph{bar}". > - Move point inside "foo" and set the mark (C-SPC). Note that this > does not "activate the region". > - TeX-fold-buffer (C-c C-o C-b). > - Move the point into

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Arash Esbati
Paul Nelson writes: > See attached, and let me know if I missed anything else. Thanks for your swift response. I installed the change (e4def30851), and closing. Best, Arash ___ bug-auctex mailing list bug-auctex@gnu.org

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Paul Nelson
See attached, and let me know if I missed anything else. Thanks, best, Paul 0001-Make-preview-placement-more-flexible.patch Description: Binary data ___ bug-auctex mailing list bug-auctex@gnu.org https://lists.gnu.org/mailman/listinfo/bug-auctex

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Paul Nelson
Hi Arash, sorry, must have messed up - I’ll be happy to redo and resend it. Thanks, best, Paul ___ bug-auctex mailing list bug-auctex@gnu.org https://lists.gnu.org/mailman/listinfo/bug-auctex

bug#70348: [PATCH] Make preview placement more flexible

2024-04-17 Thread Arash Esbati
Hi Paul, Paul Nelson writes: > This patch has no effect on current behavior, but introduces a > variable preview-find-end-function that would allow (e.g.) an external > package to modify how previews overlays are placed, without resorting > to advice. > > The code needed to be rearranged a bit

bug#58005: 13.1.4; Can't work with indirect buffers

2024-04-17 Thread Ikumi Keita
Hi Vangelis, > Vangelis Evangelou writes: > ./tex.el:2440: (and buffer-file-name > ./tex.el:2441: (file-name-directory buffer-file-name))) > In the above, I think that part of the code is in fact redundant. That's > because if the function's `expand-file-name' second