Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Scott Kostyshak
On Wed, May 02, 2018 at 08:40:14PM +, Pavel Sanda wrote: > Pavel Sanda wrote: > > > go ahead and commit your current code and I will make the extension. > > > > Ok, will commit once back home. > > It's in the tree, feel free to modify or kill the routines for what you > need... OK sounds

Hide Changes in LyX Display

2018-05-02 Thread Richard Kimberly Heck
JMarc's changes to the painting routines seem to have made it very easy now to implement an oft-requested feature: to be able to hide changes in the LyX display. The attached patch does this. There is at least one bug: If an entire paragraph is deleted, it shows as an empty line. I'm not sure how

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2018-05-02 Thread Richard Kimberly Heck
On 05/02/2018 11:35 AM, Scott Kostyshak wrote: > On Wed, May 02, 2018 at 03:17:27PM +, Jean-Marc Lasgouttes wrote: >> Le 02/05/2018 à 17:12, Scott Kostyshak a écrit : >>> On Wed, May 02, 2018 at 09:45:41AM +, Jean-Marc Lasgouttes wrote: Le 30/04/2018 à 02:45, Scott Kostyshak a écrit :

Re: [LyX/master] Update scripts to support simultaneously python 2 and 3

2018-05-02 Thread Richard Kimberly Heck
On 05/02/2018 04:45 AM, Pavel Sanda wrote: > JosĂŠ Matos wrote: >> commit 5b160e82be3797ae57632174f2510a132a7558a1 >> Author: José Matos >> Date: Sat Apr 28 10:51:35 2018 +0100 >> >> Update scripts to support simultaneously python 2 and 3 >> >> The fixes are simple

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Pavel Sanda wrote: > > go ahead and commit your current code and I will make the extension. > > Ok, will commit once back home. It's in the tree, feel free to modify or kill the routines for what you need... Pavel

Re: #11130: Lyx 2.3 is not exporting menus to Appmenu correctly (Linux)

2018-05-02 Thread Adrián Tovar Simoncic
Sorry for the delay: I wasn't very successful at digging further into the issue. Trying with several versions on different DE may be missleading, still, as far as I know, there all use vala-app-menu for their global-menu applets. Insights until today: - I was wrong, also older versions

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Scott Kostyshak wrote: > current call faster on large documents. e.g. you could change > > cur.countInsetsInSelection(GRAPHICS_CODE)>1) > > to > > cur.countInsetsInSelection(GRAPHICS_CODE, 2) == 2 > > If you are not having fun, I would be happy to make this change (but That would be nice.

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2018-05-02 Thread Scott Kostyshak
On Wed, May 02, 2018 at 03:17:27PM +, Jean-Marc Lasgouttes wrote: > Le 02/05/2018 à 17:12, Scott Kostyshak a écrit : > > On Wed, May 02, 2018 at 09:45:41AM +, Jean-Marc Lasgouttes wrote: > > > Le 30/04/2018 à 02:45, Scott Kostyshak a écrit : > > > > I don't think needauth is needed in the

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Scott Kostyshak
On Wed, May 02, 2018 at 01:48:49PM +, Pavel Sanda wrote: > Pavel Sanda wrote: > > Jean-Marc Lasgouttes wrote: > > >> Would this patch make you happy? > > >> The routine itself could be used for different type of insets if you > > >> intend to make a crussade against useless items in context

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2018-05-02 Thread Jean-Marc Lasgouttes
Le 02/05/2018 à 17:12, Scott Kostyshak a écrit : On Wed, May 02, 2018 at 09:45:41AM +, Jean-Marc Lasgouttes wrote: Le 30/04/2018 à 02:45, Scott Kostyshak a écrit : I don't think needauth is needed in the above cases. These converters are just for exporting the code in the .lyx file, not

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2018-05-02 Thread Scott Kostyshak
On Wed, May 02, 2018 at 09:45:41AM +, Jean-Marc Lasgouttes wrote: > Le 30/04/2018 à 02:45, Scott Kostyshak a écrit : > > I don't think needauth is needed in the above cases. These converters > > are just for exporting the code in the .lyx file, not for running that > > code. Unless we think

Re: Why is linguistics.lyx in merged manuals?

2018-05-02 Thread Scott Kostyshak
On Wed, May 02, 2018 at 08:58:27AM +, Pavel Sanda wrote: > Do whatever you wish, merged manuals was unsuccesfull attempt to put all of > our > manuals to Web, so our manuals are indexed by search engines. It never made it > because of xhtml export issues. OK I will just remove the note that

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Pavel Sanda wrote: > Jean-Marc Lasgouttes wrote: > >> Would this patch make you happy? > >> The routine itself could be used for different type of insets if you > >> intend to make a crussade against useless items in context menu. > > > > Shouldn't it appear only if there are at least two graphics

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: >> Would this patch make you happy? >> The routine itself could be used for different type of insets if you >> intend to make a crussade against useless items in context menu. > > Shouldn't it appear only if there are at least two graphics insets? Like this. I can keep

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Jean-Marc Lasgouttes
Le 02/05/2018 à 15:19, Pavel Sanda a écrit : Pavel Sanda wrote: Scott Kostyshak wrote: + OptItem "Unify Graphics Groups|U" "graphics-unify" Do we want to add this unconditionally to the context menu? For example, start a new document, type "hello", select it, right-click, and

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Pavel Sanda wrote: > Scott Kostyshak wrote: > > > + OptItem "Unify Graphics Groups|U" "graphics-unify" > > > > Do we want to add this unconditionally to the context menu? For example, > > start a new document, type "hello", select it, right-click, and you will > > see that "Unify Graphics

Re: SIGSEGV when copy/pasting a child doc

2018-05-02 Thread Jean-Marc Lasgouttes
Le 28/04/2018 à 00:06, Scott Kostyshak a écrit : On Wed, Apr 25, 2018 at 06:41:36PM +, Scott Kostyshak wrote: it is not 100% reproducible for me. I wonder if valgrind might be useful to try in this situation. Attached is a valgrind log. Do the invalid reads provide any clues? Does this

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: > Le 29/04/2018 ? 22:46, Scott Kostyshak a écrit : >> There are other irrelevant options (e.g. >> "Accept Change"), but Vincent (7108b8e4) is not around for me to complain >> to >> about it :). > > Could you elaborate? I do not see it after typing hello in an empty >

Re: [LyX/master] Add 'needauth' option to converters that need explicit user authorization.

2018-05-02 Thread Jean-Marc Lasgouttes
Le 30/04/2018 à 02:45, Scott Kostyshak a écrit : I don't think needauth is needed in the above cases. These converters are just for exporting the code in the .lyx file, not for running that code. Unless we think there is a way to to trick lyxknitr.R (and e.g. run arbitrary code). But I don't

Re: Why is linguistics.lyx in merged manuals?

2018-05-02 Thread Pavel Sanda
Scott Kostyshak wrote: > It causes the following terminal messages: > > step: Counter does not exist: examplei > step: Counter does not exist: examplei > step: Counter does not exist: subexamplei > step: Counter does not exist: subexamplei > step: Counter does not exist: subexamplei >

Re: Inset border painting issue (regression)

2018-05-02 Thread Jean-Marc Lasgouttes
Le 02/05/2018 à 10:52, Pavel Sanda a écrit : Result: when the cursor is blinking (i.e. when it disappears), the right border of the LyX note is not redrawn. Can anyone else reproduce? Is it this one: https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg204745.html ? Pavel Yes, it looks

Re: Inset border painting issue (regression)

2018-05-02 Thread Pavel Sanda
Scott Kostyshak wrote: > To reproduce (e.g. on current master): > > 1. Open the attached two .lyx files in the same LyX instance. > 2. Place the cursor at the end of the mwe2.lyx. > 3. Press once (this will enter the Note inset). > 4. Press (this will delete the "e"). > > Result: when the

Re: [LyX/master] Update scripts to support simultaneously python 2 and 3

2018-05-02 Thread Pavel Sanda
JosĂŠ Matos wrote: > commit 5b160e82be3797ae57632174f2510a132a7558a1 > Author: José Matos > Date: Sat Apr 28 10:51:35 2018 +0100 > > Update scripts to support simultaneously python 2 and 3 > > The fixes are simple and on line with the changes made during > the

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Jean-Marc Lasgouttes
Le 29/04/2018 à 22:46, Scott Kostyshak a écrit : There are other irrelevant options (e.g. "Accept Change"), but Vincent (7108b8e4) is not around for me to complain to about it :). Could you elaborate? I do not see it after typing hello in an empty document. JMarc

Re: [LyX/master] Adapt caret height to context in mathed.

2018-05-02 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: >>> Pavel, please test this new code. It should get the cursor size right >>> without crashing of changing bizarrely the current font. >>> >>> Riki, if this works, I will probably propose to backport to 2.3.2. >> Any chance you would provide 2.3 patch? Anything in

Re: [LyX/master] Unify graphics-groups inside marked block functionality.

2018-05-02 Thread Pavel Sanda
Scott Kostyshak wrote: > > + OptItem "Unify Graphics Groups|U" "graphics-unify" > > Do we want to add this unconditionally to the context menu? For example, > start a new document, type "hello", select it, right-click, and you will > see that "Unify Graphics Groups" is an option even