On Mon, Apr 11, 2022 at 02:17:44PM +0200, Jean-Marc Lasgouttes wrote: > Le 11/04/2022 ?? 13:44, Pavel Sanda a écrit : > >So what is your reading of the results? > >Seems to me no interest. As a result I would favor committing my patch > >minus context menu UI plus any changes you might have for the wording. > >And I'll add some note into manuals. > > I did read your patch, but I do not know what the use case is.
My use case is opening/closing all branch insets of a given name. > For a given branch, is closes open branches and opens closed ones? Setting > them to some preferred state (active=open) would seem more useful to me. I bound the lfun to branch inset context menu so toggling made sense. But I can make the lfun more generic, i.e. allowing open/close/toggle arguments. > >The other exchange with JMarc seem to go different direction if I understand > >it right... > > I am not totally sure, since I think we can extend a bit the semantics of > inset-toggle to make your patch actually useful. We would need to pass the "correct" branch name into lfun call. You want inset-forall to sniff around for proper name? Does not look right. Or is there way how to pass that string via specific parameter in context menu definition? Pavel -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel
