Mosè Giordano <[email protected]> writes:
>> Yes, I'll have a look but probably not today anymore.
>
> Well, probably having the patch really attached would be better ;-)
Ah, I've thought that was an incapability of my mobile email client. :-)
Anyway, with respect to the patch: I don't like that the "Evince" viewer
entry might call either Evince or Atril depending on the value of
`TeX-evince-gnome'. Mate users don't need to know that Atril is a fork
of Evince. And maybe some users may want to use both Evince and Atril.
So I'd rather have two separate viewer entries for Evince and Atril, and
those would call `TeX-evince-dbus-p' with the right app and desktop
environment name, and then return `TeX-evince-sync-view' or
`TeX-atril-sync-view' where these two are just
(defun TeX-evince-sync-view ()
(TeX-evince-sync-view-1 "evince" "gnome"))
(defun TeX-atril-sync-view ()
(TeX-evince-sync-view-1 "atril" "mate"))
and `TeX-evince-sync-view-1' is the `TeX-evince-sync-view' from your
patch with 2 additional args providing the information of the now
unneeded `TeX-evince-gnome'.
WDYT?
Bye,
Tassilo
_______________________________________________
bug-auctex mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/bug-auctex
