Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Enrico Forestieri
On Fri, Dec 21, 2007 at 05:55:41PM -0600, Bo Peng wrote: > > This patch avoids the crash. However, note that previously the > > untranslatable character was actually correctly shown in the > > ViewSource window and a warning was output to the console. > > Could the old behavior be restored? > > >

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Bo Peng
> This patch avoids the crash. However, note that previously the > untranslatable character was actually correctly shown in the > ViewSource window and a warning was output to the console. > Could the old behavior be restored? > I do not like the old behavior at all. The problematic character is h

math, auto, etc. in toolbar.ui

2007-12-21 Thread Stefan Schimanski
Hi! I am trying to make sense of the toolbar.ui logic. In ToolbarBackend::readToolbarSettings there is no support for an "auto" flag. But ToolbarInfo::Flags suggests that there is a need to set this bit to enable the auto mode from the default configuration: AUTO = 512, //< only if AUTO

Re: CharStyle metrics broken in 1.5.3

2007-12-21 Thread Abdelrazak Younes
Enrico Forestieri wrote: On Fri, Dec 21, 2007 at 09:31:24PM +0100, Abdelrazak Younes wrote: Enrico Forestieri wrote: Load the attached .lyx file using 1.5.3 or 1.5.4svn and move the cursor inside the inset to see it. What do you mean exactly by broken? I can see that the cursor looks like it

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Enrico Forestieri
On Sat, Dec 22, 2007 at 02:09:40PM +1800, Bo Peng wrote: > > > > So please commit and close bug 3511 with target 1.5.4. > > > > > > Done. > > > > Note that now LyX crashes on View->Source. > > Ohmm, This is because view source does not catch this exception. How > about the attached patch? It is n

Re: CharStyle metrics broken in 1.5.3

2007-12-21 Thread Enrico Forestieri
On Fri, Dec 21, 2007 at 09:31:24PM +0100, Abdelrazak Younes wrote: > Enrico Forestieri wrote: > > Load the attached .lyx file using 1.5.3 or 1.5.4svn and move the > > cursor inside the inset to see it. > > What do you mean exactly by broken? I can see that the cursor looks like > it is inside th

Re: Abort on Close: 1.6.svn

2007-12-21 Thread Abdelrazak Younes
rgheck wrote: Abdelrazak Younes wrote: rgheck wrote: FileName.cpp(194): Could not move file /tmp/lyx_tmpdir27404xpQ0V/lyxauto4095QMqYlq to /home/rgheck#newfile1.lyx# lyx: xcb_io.c:182: process_responses: Assertion `((int) (((dpy->last_request_read)) - ((dpy->request))) <= 0)' failed. Looks

Re: Scrolling Bug

2007-12-21 Thread rgheck
Abdelrazak Younes wrote: rgheck wrote: Nice work on lots of this, Abdel, Glad you like it :-) but on File>New right now I get the cursor off the top of the screen. This is on Fedora 8, Qt 4.3.2. Fixed. Thanks as always... rh

Re: Abort on Close: 1.6.svn

2007-12-21 Thread rgheck
Abdelrazak Younes wrote: rgheck wrote: FileName.cpp(194): Could not move file /tmp/lyx_tmpdir27404xpQ0V/lyxauto4095QMqYlq to /home/rgheck#newfile1.lyx# lyx: xcb_io.c:182: process_responses: Assertion `((int) (((dpy->last_request_read)) - ((dpy->request))) <= 0)' failed. Looks like we're miss

Re: CharStyle metrics broken in 1.5.3

2007-12-21 Thread Abdelrazak Younes
Enrico Forestieri wrote: Load the attached .lyx file using 1.5.3 or 1.5.4svn and move the cursor inside the inset to see it. What do you mean exactly by broken? I can see that the cursor looks like it is inside the inset when in front of it; is it what you mean? Abdel.

Re: Included Math Macros

2007-12-21 Thread Abdelrazak Younes
Stefan Schimanski wrote: I have several files that include another file, ggmacros.lyx, that includes all the math macros I need for most of my papers. In 1.5.x, and previously, so long as the macro file was loaded before the base file, the macros would be seen and used by LyX. In the current tr

Re: Scrolling Bug

2007-12-21 Thread Abdelrazak Younes
rgheck wrote: Nice work on lots of this, Abdel, Glad you like it :-) but on File>New right now I get the cursor off the top of the screen. This is on Fedora 8, Qt 4.3.2. Fixed. Abdel.

Re: CharStyle metrics broken in 1.5.3

2007-12-21 Thread Abdelrazak Younes
Enrico Forestieri wrote: Load the attached .lyx file using 1.5.3 or 1.5.4svn and move the cursor inside the inset to see it. This is a side effect of r21428. Should be easy to fix, Abdel. I'll have a look. Abdel.

Re: Abort on Close: 1.6.svn

2007-12-21 Thread Abdelrazak Younes
rgheck wrote: FileName.cpp(194): Could not move file /tmp/lyx_tmpdir27404xpQ0V/lyxauto4095QMqYlq to /home/rgheck#newfile1.lyx# lyx: xcb_io.c:182: process_responses: Assertion `((int) (((dpy->last_request_read)) - ((dpy->request))) <= 0)' failed. Looks like we're missing a "/". Could you give

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Bo Peng
> > > So please commit and close bug 3511 with target 1.5.4. > > > > Done. > > Note that now LyX crashes on View->Source. Ohmm, This is because view source does not catch this exception. How about the attached patch? It is not a good idea to show the error dialog here because the view source dialo

Re: Included Math Macros

2007-12-21 Thread rgheck
Stefan Schimanski wrote: I have several files that include another file, ggmacros.lyx, that includes all the math macros I need for most of my papers. In 1.5.x, and previously, so long as the macro file was loaded before the base file, the macros would be seen and used by LyX. In the current tr

Re: Included Math Macros

2007-12-21 Thread Stefan Schimanski
I have several files that include another file, ggmacros.lyx, that includes all the math macros I need for most of my papers. In 1.5.x, and previously, so long as the macro file was loaded before the base file, the macros would be seen and used by LyX. In the current trunk, however, that is

Re: [PATCH] Spacing Metrics

2007-12-21 Thread rgheck
Enrico Forestieri wrote: This doesn't seem to be correct. Here is how those spaces are defined in LaTeX: \def\enspace{\kern.5em } \def\enskip{\hskip.5em\relax} \def\quad{\hskip1em\relax} \def\qquad{\hskip2em\relax} I've made the spaces match this, more or less, using width(char_type('M')) f

Included Math Macros

2007-12-21 Thread rgheck
I have several files that include another file, ggmacros.lyx, that includes all the math macros I need for most of my papers. In 1.5.x, and previously, so long as the macro file was loaded before the base file, the macros would be seen and used by LyX. In the current trunk, however, that is n

Re: Alignment in Gather Environment

2007-12-21 Thread rgheck
I think that the attached patch fixes it. Confirmed that Enrico's patch fixes this. Jürgen, Ok for branch? I'd suggest it. rh

Navigation Does Not Work in Trunk

2007-12-21 Thread rgheck
Open the User's Guide. Do Navigate>Whatever. Nada. rh

Abort on Close: 1.6.svn

2007-12-21 Thread rgheck
FileName.cpp(194): Could not move file /tmp/lyx_tmpdir27404xpQ0V/lyxauto4095QMqYlq to /home/rgheck#newfile1.lyx# lyx: xcb_io.c:182: process_responses: Assertion `((int) (((dpy->last_request_read)) - ((dpy->request))) <= 0)' failed. Looks like we're missing a "/". rh

Scrolling Bug

2007-12-21 Thread rgheck
Nice work on lots of this, Abdel, but on File>New right now I get the cursor off the top of the screen. This is on Fedora 8, Qt 4.3.2. rh

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Enrico Forestieri
On Sat, Dec 22, 2007 at 11:33:23AM +1800, Bo Peng wrote: > > So please commit and close bug 3511 with target 1.5.4. > > Done. Note that now LyX crashes on View->Source. -- Enrico

About scrolling in 1.6

2007-12-21 Thread Abdelrazak Younes
Dear all, As a Christmas gift I decided to do something about the erratic scrolling. The result is now in SVN. I used the simple approach that I outlined sometime ago in an exchange with Tomaso. Well, even if simple and not very intrusive this change required quite some thinking and testing.

Re: [LyX wiki] recent notify posts

2007-12-21 Thread Abdelrazak Younes
Someone hacked the wiki with pretty bad links.. Abdel. Apache wrote: All recent posts: http://wiki.lyx.org/Site/AllRecentChanges * http://wiki.lyx.org/FAQ.ERT . . . 2007-12-21 17:00 CET by Santuhaster

CharStyle metrics broken in 1.5.3

2007-12-21 Thread Enrico Forestieri
Load the attached .lyx file using 1.5.3 or 1.5.4svn and move the cursor inside the inset to see it. This is a side effect of r21428. Should be easy to fix, Abdel. -- Enrico #LyX 1.5.3 created this file. For more info see http://www.lyx.org/ \lyxformat 276 \begin_document \begin_header \textclass

Re: Alignment in Gather Environment

2007-12-21 Thread Enrico Forestieri
On Thu, Dec 20, 2007 at 10:04:21PM -0500, rgheck wrote: > Enrico Forestieri wrote: > > On Fri, Dec 21, 2007 at 12:08:18AM +0100, Jean-Marc Lasgouttes wrote: > > > > > >> rgheck <[EMAIL PROTECTED]> writes: > >> > >> > >>> In the gather environment in LyX 1.5.x, as well as in earlier > >>> v

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Bo Peng
> So please commit and close bug 3511 with target 1.5.4. Done. Bo

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Jürgen Spitzmüller
Bo Peng wrote: > Right now, the exception will stop the export process so only the > first unknown character is reported. If we want to report all of them, > ErrorList has to be passed to a deeper level which would lead to a > more intrusive patch. I chose the simpler solution because I think it >

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Bo Peng
> I just tested it. It looks good. One thing I noted: If the document contains > multiple unknown characters, only the first one is reported by the error > dialog. Is this intended? Right now, the exception will stop the export process so only the first unknown character is reported. If we want to

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Jürgen Spitzmüller
Bo Peng wrote: > Jurgen, > > Is it OK to apply this patch to the branch now? I just tested it. It looks good. One thing I noted: If the document contains multiple unknown characters, only the first one is reported by the error dialog. Is this intended? Jürgen

Re: [PATCH] Use ErrorList to handle encoding error.

2007-12-21 Thread Bo Peng
On Dec 19, 2007 11:54 AM, Bo Peng <[EMAIL PROTECTED]> wrote: > > The patch is for 1.5.4, please test. I will commit after a few days of > > using it. Jurgen, Is it OK to apply this patch to the branch now? Bo

Re: [Bug 4147] Multiple cells paste as one in tables

2007-12-21 Thread Jürgen Spitzmüller
Bennett Helm wrote: > You mean matrices? -- I did this, and with 1.5.3 it works, Good. > but with some differences from how your patch works in tables. Yes, but this is yet another story. Jürgen

Re: [Bug 4147] Multiple cells paste as one in tables

2007-12-21 Thread Bennett Helm
On Dec 21, 2007, at 11:09 AM, Jürgen Spitzmüller wrote: I don't know what's the status wrt math grids. Bennett, does copying from math grids work? You mean matrices? -- I did this, and with 1.5.3 it works, but with some differences from how your patch works in tables. In a 5x5 matrix, cu

Re: [Bug 4147] Multiple cells paste as one in tables

2007-12-21 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > Why is this needed only for tabular? In text, pasteFromStack is used if the condition fails, which just works apparently: case LFUN_PASTE: cur.message(_("Paste")); cap::replaceSelection(cur); if (cmd.argument()

Re: [Bug 4147] Multiple cells paste as one in tables

2007-12-21 Thread Jean-Marc Lasgouttes
Jürgen Spitzmüller <[EMAIL PROTECTED]> writes: > Juergen Spitzmueller wrote: >> So LFUN_PASTE for multiple tabular cells isn't supposed to work both for >> Mac and Windows. > > The attached patch fixes it, according to Bennett's testing. The patch is > against branch. Why is this needed only for

Re: [Bug 4147] Multiple cells paste as one in tables

2007-12-21 Thread Jürgen Spitzmüller
Juergen Spitzmueller wrote: > So LFUN_PASTE for multiple tabular cells isn't supposed to work both for > Mac and Windows. The attached patch fixes it, according to Bennett's testing. The patch is against branch. Opinions? Jürgen Index: src/insets/InsetTabular.cpp ===

Re: [PATCH] Spacing Metrics

2007-12-21 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > You can take the width of the 'm' letter, which was the original > definition. 'M' (capital), for that matter. However, this never was the original definition. The original definition refers to the height of the body (the metal block of the letter). > I read some

Re: [Cvslog] r22216 [2/2] - in /lyx-devel/trunk/po: LINGUAS ru.po

2007-12-21 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > Juergen, I got an updated ru.po for trunk and I would like to use it > for branch. This means readding ru to LINGUAS (we removed it earlier). > OK? Yes, fine. Jürgen

Re: [PATCH] Spacing Metrics

2007-12-21 Thread Jürgen Spitzmüller
rgheck wrote: > Jurgen, would you want it for branch? Yes. Jürgen

Re: [PATCH] Spacing Metrics

2007-12-21 Thread Jürgen Spitzmüller
rgheck wrote: > Can we find out how big an em is? An em is exactly the font size. So for a 10pt font, 1 em = 10 pt. Roughly, it also matches the width of the letter "M". Jürgen

Re: [PATCH] Spacing Metrics

2007-12-21 Thread Jean-Marc Lasgouttes
rgheck <[EMAIL PROTECTED]> writes: > No, I doubt they're really right, but they look better than what we > had, which fixes the width of a quad e.g. independently of the font > size. So: Can we find out how big an em is? Is there some fixed > relationship between the size of an em and the size of

Re: scroll crash

2007-12-21 Thread Abdelrazak Younes
Leuven, E. wrote: when scrolling down (and up) in the user guide: Year, some bugs remain, I will solve them one by one. Please continue testing (and try to fix if you can ;-) Abdel.

scroll crash

2007-12-21 Thread Leuven, E.
when scrolling down (and up) in the user guide: lyx.exe!std::vector >::operator[](unsigned int _Pos=1680) Line 756 + 0x17 bytesC++ lyx.exe!lyx::BufferView::updateScrollbar() Line 452 + 0x1f bytes C++ lyx.exe!lyx::BufferView::draw(lyx::frontend::Painter & pain={...}) Line 1906