Re: Naming custom insets

2010-08-11 Thread Jean-Marc LASGOUTTES
Richard Heck rgh...@comcast.net writes: Next problem: Reversion is impossible. We have no way of knowing if \begin_inset Flex MyStyle should be reverted to \begin_inset Flex Custom:MyStyle or to \begin_inset Flex CharStyle:MyStyle or not at all. Very good point. One solution

Re: changeset/35060 - color regressions

2010-08-11 Thread Jean-Marc LASGOUTTES
Uwe Stöhr uwesto...@web.de writes: I don't see that LyX is so special. It uses its own colors like nearly every program I have installed on my Windows. You mean word, openoffice, firefox, chrome, outlook, photoshop? I understand that many programs like weird colors and round windows (like

Re: r35118 - lyx-devel/trunk/lib/layouts

2010-08-11 Thread Jean-Marc LASGOUTTES
Jürgen Spitzmüller sp...@lyx.org writes: Log: Fix a syntax error in the class declaration. This created an error in configure.py, without any good explanation. Annoying. Thanks to Jean-Pierre for spotting it. please backport if needed. No backport needed. It was a recent bug (the big

Re: r35109 - lyx-devel/trunk/src

2010-08-11 Thread Jean-Marc LASGOUTTES
lasgout...@lyx.org writes: Author: lasgouttes Date: Mon Aug 9 22:37:31 2010 New Revision: 35109 URL: http://www.lyx.org/trac/changeset/35109 Log: fix bug #6786: InsetLayout name containing spaces is incompatible with keybindings/toolbars Juergen, I think this one belongs to branch. It

Re: r35109 - lyx-devel/trunk/src

2010-08-11 Thread Jürgen Spitzmüller
Jean-Marc LASGOUTTES wrote: Author: lasgouttes Date: Mon Aug 9 22:37:31 2010 New Revision: 35109 URL: http://www.lyx.org/trac/changeset/35109 Log: fix bug #6786: InsetLayout name containing spaces is incompatible with keybindings/toolbars Juergen, I think this one belongs to

Re: platex as the default latex command?

2010-08-11 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: No. latex is found and ascribed to the LATEX variable. But later (line 215), is is replaced by platex (given that platex was found): if cmdOutput(PLATEX + ' chklatex.ltx').find('pLaTeX2e') != -1: # We have the Japanese pLaTeX2e

Branch Bug

2010-08-11 Thread Richard Heck
Create a footnote. Now type this \emph{is} weird. Here, the is is larger than this and so is weird. rh

Re: Branch Bug

2010-08-11 Thread Jean-Marc Lasgouttes
Le 11 août 10 à 20:38, Richard Heck a écrit : Create a footnote. Now type this \emph{is} weird. Here, the is is larger than this and so is weird. I can't reproduce. By \emph, do you mean emph font? Maybe an exempl file would help. JMarc

Spellcheck a whole row

2010-08-11 Thread Stephan Witt
Hi, I've prepared the following patch to demonstrate the way to speed up the native spell checker on mac. In case of spell error it is as slow as the original version. I'd like to improve that next... Stephan Index: src/AppleSpellChecker.h

Re: Branch Bug

2010-08-11 Thread Richard Heck
On 08/11/2010 02:47 PM, Jean-Marc Lasgouttes wrote: Le 11 août 10 à 20:38, Richard Heck a écrit : Create a footnote. Now type this \emph{is} weird. Here, the is is larger than this and so is weird. I can't reproduce. By \emph, do you mean emph font? Maybe an exempl file would help. Hmm.

Re: Spellcheck a whole row

2010-08-11 Thread Jean-Marc Lasgouttes
Le 11 août 10 à 21:21, Stephan Witt a écrit : I've prepared the following patch to demonstrate the way to speed up the native spell checker on mac. In case of spell error it is as slow as the original version. I'd like to improve that next... I do not have time to test it, so I'll just make

Re: Naming custom insets

2010-08-11 Thread Jean-Marc Lasgouttes
We can also add code to LyX 1.6 that reads these borked files: if MyStyle does not exist and the inset is of type Custom, try Custom:MyStyle. Of course, we do not know that the inset is of type custom... So, we would have to try in turn CharStyle:MyStyle, Element:MyStyle and Custom:MyStyle.

Re: Naming custom insets

2010-08-11 Thread Richard Heck
On 08/11/2010 04:35 PM, Jean-Marc Lasgouttes wrote: We can also add code to LyX 1.6 that reads these borked files: if MyStyle does not exist and the inset is of type Custom, try Custom:MyStyle. Of course, we do not know that the inset is of type custom... So, we would have to try in turn

Re: Spellcheck a whole row

2010-08-11 Thread Abdelrazak Younes
On Wed, Aug 11, 2010 at 9:21 PM, Stephan Witt st.w...@gmx.net wrote: Hi, I've prepared the following patch to demonstrate the way to speed up the native spell checker on mac. In case of spell error it is as slow as the original version. I'd like to improve that next... Please don't

Re: Naming custom insets

2010-08-11 Thread Jean-Marc Lasgouttes
By the way, I sometimes feel hamstrung by the thought that files produced with LyX 2.0 ought to work just as well in LyX 1.2.x. Yes, I refuse to abandon a cleanup just because we would not be able to revert. Personally, I would go through the route of hardcoded list of insets. But in case

Re: Spellcheck a whole row

2010-08-11 Thread Jean-Marc Lasgouttes
Le 11 août 10 à 23:47, Abdelrazak Younes a écrit : Please don't introduce Row.h dependency into Paragraph. The first is about screen and the second about contents. Just make a new Paragraph::spellCheckRange(ps_type from, pos_type to) And I agree with JMarc that this method is too complicated

Re: Naming custom insets

2010-08-11 Thread Jean-Marc LASGOUTTES
Richard Heck writes: > Next problem: Reversion is impossible. We have no way of knowing if > \begin_inset Flex MyStyle > should be reverted to > \begin_inset Flex Custom:MyStyle > or to > \begin_inset Flex CharStyle:MyStyle > or not at all. Very good point. One

Re: changeset/35060 - color regressions

2010-08-11 Thread Jean-Marc LASGOUTTES
Uwe Stöhr writes: > I don't see that LyX is so special. It uses its own colors like nearly > every program I have installed on my Windows. You mean word, openoffice, firefox, chrome, outlook, photoshop? I understand that many programs like weird colors and round windows

Re: r35118 - lyx-devel/trunk/lib/layouts

2010-08-11 Thread Jean-Marc LASGOUTTES
Jürgen Spitzmüller writes: >> Log: >> Fix a syntax error in the class declaration. This created an error in >> configure.py, without any good explanation. Annoying. >> >> Thanks to Jean-Pierre for spotting it. > > please backport if needed. No backport needed. It was a recent bug

Re: r35109 - lyx-devel/trunk/src

2010-08-11 Thread Jean-Marc LASGOUTTES
lasgout...@lyx.org writes: > Author: lasgouttes > Date: Mon Aug 9 22:37:31 2010 > New Revision: 35109 > URL: http://www.lyx.org/trac/changeset/35109 > > Log: > fix bug #6786: InsetLayout name containing spaces is incompatible with > keybindings/toolbars Juergen, I think this one belongs to

Re: r35109 - lyx-devel/trunk/src

2010-08-11 Thread Jürgen Spitzmüller
Jean-Marc LASGOUTTES wrote: > > Author: lasgouttes > > Date: Mon Aug 9 22:37:31 2010 > > New Revision: 35109 > > URL: http://www.lyx.org/trac/changeset/35109 > > > > Log: > > fix bug #6786: InsetLayout name containing spaces is incompatible with > > keybindings/toolbars > > Juergen, I think

Re: platex as the default latex command?

2010-08-11 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > > No. "latex" is found and ascribed to the LATEX variable. But later > > (line 215), > > > > is is replaced by "platex" (given that platex was found): > > if cmdOutput(PLATEX + ' chklatex.ltx').find('pLaTeX2e') != -1: > > # We have the Japanese

Branch Bug

2010-08-11 Thread Richard Heck
Create a footnote. Now type "this \emph{is} weird". Here, the "is" is larger than "this" and so is "weird". rh

Re: Branch Bug

2010-08-11 Thread Jean-Marc Lasgouttes
Le 11 août 10 à 20:38, Richard Heck a écrit : Create a footnote. Now type "this \emph{is} weird". Here, the "is" is larger than "this" and so is "weird". I can't reproduce. By \emph, do you mean emph font? Maybe an exempl file would help. JMarc

Spellcheck a whole row

2010-08-11 Thread Stephan Witt
Hi, I've prepared the following patch to demonstrate the way to speed up the native spell checker on mac. In case of spell error it is as slow as the original version. I'd like to improve that next... Stephan Index: src/AppleSpellChecker.h

Re: Branch Bug

2010-08-11 Thread Richard Heck
On 08/11/2010 02:47 PM, Jean-Marc Lasgouttes wrote: Le 11 août 10 à 20:38, Richard Heck a écrit : Create a footnote. Now type "this \emph{is} weird". Here, the "is" is larger than "this" and so is "weird". I can't reproduce. By \emph, do you mean emph font? Maybe an exempl file would help.

Re: Spellcheck a whole row

2010-08-11 Thread Jean-Marc Lasgouttes
Le 11 août 10 à 21:21, Stephan Witt a écrit : I've prepared the following patch to demonstrate the way to speed up the native spell checker on mac. In case of spell error it is as slow as the original version. I'd like to improve that next... I do not have time to test it, so I'll just make

Re: Naming custom insets

2010-08-11 Thread Jean-Marc Lasgouttes
We can also add code to LyX 1.6 that reads these borked files: if MyStyle does not exist and the inset is of type Custom, try Custom:MyStyle. Of course, we do not know that the inset is of type custom... So, we would have to try in turn CharStyle:MyStyle, Element:MyStyle and Custom:MyStyle.

Re: Naming custom insets

2010-08-11 Thread Richard Heck
On 08/11/2010 04:35 PM, Jean-Marc Lasgouttes wrote: We can also add code to LyX 1.6 that reads these borked files: if MyStyle does not exist and the inset is of type Custom, try Custom:MyStyle. Of course, we do not know that the inset is of type custom... So, we would have to try in turn

Re: Spellcheck a whole row

2010-08-11 Thread Abdelrazak Younes
On Wed, Aug 11, 2010 at 9:21 PM, Stephan Witt wrote: > Hi, > > I've prepared the following patch to demonstrate the way to speed up the > native spell checker on mac. In case of spell error it is as slow as the > original version. I'd like to improve that next... > Please don't

Re: Naming custom insets

2010-08-11 Thread Jean-Marc Lasgouttes
By the way, I sometimes feel hamstrung by the thought that files produced with LyX 2.0 ought to work just as well in LyX 1.2.x. Yes, I refuse to abandon a cleanup just because we would not be able to revert. Personally, I would go through the route of hardcoded list of insets. But in case

Re: Spellcheck a whole row

2010-08-11 Thread Jean-Marc Lasgouttes
Le 11 août 10 à 23:47, Abdelrazak Younes a écrit : Please don't introduce Row.h dependency into Paragraph. The first is about screen and the second about contents. Just make a new Paragraph::spellCheckRange(ps_type from, pos_type to) And I agree with JMarc that this method is too complicated