Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-25 Thread Jean-Marc Lasgouttes
Le 20/07/2017 à 15:13, Jürgen Spitzmüller a écrit : Am Donnerstag, den 20.07.2017, 14:57 +0200 schrieb Jean-Marc Lasgouttes: Technically, insets with a dialog are also "editable". No, now we say that they "have settings". The notion of editable is really related to entering in the inset with

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jürgen Spitzmüller
Am Donnerstag, den 20.07.2017, 14:57 +0200 schrieb Jean-Marc Lasgouttes: > > Technically, insets with a dialog are also "editable". > > No, now we say that they "have settings". The notion of editable is > really related to entering in the inset with a cursor. I see. Didn't know. This strikes

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2017 à 14:08, Jürgen Spitzmüller a écrit : Am Donnerstag, den 20.07.2017, 12:32 +0200 schrieb Jean-Marc Lasgouttes: Le 20/07/2017 à 11:52, Jürgen Spitzmüller a écrit : The reason probably is that the naming of these functions is not really good. Indeed. As I understand your

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jürgen Spitzmüller
Am Donnerstag, den 20.07.2017, 12:32 +0200 schrieb Jean-Marc Lasgouttes: > Le 20/07/2017 à 11:52, Jürgen Spitzmüller a écrit : > > The reason probably is that the naming of these functions is not > > really > > good. > > Indeed. > > > As I understand your comments, shouldn't it be something

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2017 à 11:52, Jürgen Spitzmüller a écrit : Am Donnerstag, den 20.07.2017, 11:38 +0200 schrieb Jean-Marc Lasgouttes: Sigh. It turns out I understood the whole isActive/editable/nargs issue only vaguely. The reason probably is that the naming of these functions is not really good.

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2017 à 11:52, Jürgen Spitzmüller a écrit : The reason probably is that the naming of these functions is not really good. Indeed. As I understand your comments, shouldn't it be something like: cursorCanEnter() (editable()) Well technically cursor can enter a closed

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jürgen Spitzmüller
Am Donnerstag, den 20.07.2017, 11:38 +0200 schrieb Jean-Marc Lasgouttes: > Sigh. It turns out I understood the whole isActive/editable/nargs > issue > only vaguely. The reason probably is that the naming of these functions is not really good. As I understand your comments, shouldn't it be

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-20 Thread Jean-Marc Lasgouttes
Le 20/07/2017 à 03:34, Scott Kostyshak a écrit : Open the attached .lyx file. Make sure that the cursor is at the beginning and that the footnote inset is not visible on the screen. Then do a find for the word "find". Expected: that Lyx opens the inset and highlights the word "find". Actual:

Re: [LyX/master] Fixup the fixup d0acc3e57044: use editable()/isActive()

2017-07-19 Thread Scott Kostyshak
On Tue, Jun 27, 2017 at 04:47:10PM +0200, Jean-Marc Lasgouttes wrote: > commit 13c3c1485b68980c51658cef8fadf804982d75ee > Author: Jean-Marc Lasgouttes > Date: Fri Jun 23 20:32:32 2017 +0200 > > Fixup the fixup d0acc3e57044: use editable()/isActive() > > While