Re: [patch] bug 3407: Undo in mathmode

2007-06-07 Thread Alfredo Braunstein
Jean-Marc Lasgouttes wrote: > Alfredo> Should have "Mathed" in the name then no? > > No because the long term objectove is to merge all these things, I > guess. I think that the nice name should be reserved as a reward for the merger ;-) > And a bit of mystery and surprise makes life more

Re: EndEnvironment layout entry.

2007-06-07 Thread José Matos
On Thursday 07 June 2007 20:03:09 Bo Peng wrote: > > Do you want to insert an inset? I want everything to be an inset. Say an inset chapter, with the first line being the title, and all other lines being inside the inset. The same would be applied to sections and subsections and so on...

Re: EndEnvironment layout entry.

2007-06-07 Thread José Matos
On Thursday 07 June 2007 20:03:09 Bo Peng wrote: > Using the environment approach, I enter , choose separator > layout, , and then start a new list. (How to avoid that second > ?) Something like new page inset? An inset separator? > Bo -- José Abílio

Re: EndEnvironment layout entry.

2007-06-07 Thread Bo Peng
Something like new page inset? An inset separator? That still falls into the insert something category. You still have to insert in the 'standard' environment. Otherwise your inset, although output nothing in latex, will be nested. I mean 1. item1 2. your do-nothing inset 3. item1 Bo

Re: [PATCH] Allow short titles in AMS articles

2007-06-07 Thread Martin Vermeer
On Thu, Jun 07, 2007 at 12:33:45PM -0400, Paul A. Rubin wrote: > The attached patch to amsdefs.inc (versions 1.4.x and 1.5.x) turns on > short titles, which amsart.cls supports. I'm not sure why they're not > on already -- maybe there was a reason? I suppose it was because the feature was

Re: Fixing #3790, 3801, 3809 in the only right way, no further patchworks!

2007-06-07 Thread Stefan Schimanski
Here are two further patches to clean up the code: * local_bidi.patch: The bidi object in the class Text is only used in places now where a ParagraphMetrics::computeRowMetrics call comes before. And this function the Bidi object is updated. So it is a little step now to remove the

Re: [PATCH] 3782 Crash with selection of table cells

2007-06-07 Thread Alfredo Braunstein
Bo Peng wrote: >> >> saveSelection does not hurt even in the LFUN_FINISHED case. At worst >> it is useless. > > Logically speaking, we should saveSelection after cur has moved to its > place. In that case, cur is undispatched so saveSelection should not > be called. Maybe the best would be

Re: Fixing #3790, 3801, 3809 in the only right way, no further patchworks!

2007-06-07 Thread José Matos
On Thursday 07 June 2007 21:01:10 Stefan Schimanski wrote: > Here are two further patches to clean up the code: > > * local_bidi.patch: The bidi object in the class Text is only used in > places now where a ParagraphMetrics::computeRowMetrics call comes > before. And this function the Bidi object

Re: [PATCH] 3782 Crash with selection of table cells

2007-06-07 Thread Bo Peng
> Logically speaking, we should saveSelection after cur has moved to its > place. In that case, cur is undispatched so saveSelection should not > be called. Maybe the best would be calling saveSelection at the end of Cursor::dispatch (maybe adding the check if the cursor has moved), and nowhere

Re: [PATCH] 3782 Crash with selection of table cells

2007-06-07 Thread Alfredo Braunstein
Bo Peng wrote: >> > Logically speaking, we should saveSelection after cur has moved to its >> > place. In that case, cur is undispatched so saveSelection should not >> > be called. >> >> Maybe the best would be calling saveSelection at the end of >> Cursor::dispatch (maybe adding the check if the

Re: [patch] KeyMap bug (again)

2007-06-07 Thread José Matos
On Thursday 07 June 2007 08:47:24 Jean-Marc Lasgouttes wrote: > The patch is OK to me (with the extra switchKeymap). Dov if you come with a patch with this suggestion implemented then you have my OK. :-) > JMarc -- José Abílio

Re: Spaces on RTL boundaries

2007-06-07 Thread José Matos
On Thursday 07 June 2007 14:19:41 Stefan Schimanski wrote: > Order to apply: > > no_bidi_isboundary.patch > no_bidi_setcurrentfont.patch > local_bidi.patch > cursor_get_font.patch > rtl_spaces.patch What is missing to apply from this set? > Stefan -- José Abílio

Re: Spaces on RTL boundaries

2007-06-07 Thread Stefan Schimanski
Am 07.06.2007 um 22:29 schrieb José Matos: On Thursday 07 June 2007 14:19:41 Stefan Schimanski wrote: Order to apply: no_bidi_isboundary.patch no_bidi_setcurrentfont.patch local_bidi.patch cursor_get_font.patch rtl_spaces.patch What is missing to apply from this set? Just the last one.

Re: Spaces on RTL boundaries

2007-06-07 Thread José Matos
On Thursday 07 June 2007 21:41:45 Stefan Schimanski wrote: > Just the last one. Want to hear some report by people here in the   > thread. Then it can go in. That is fair. > Stefan -- José Abílio

Re: Spaces on RTL boundaries

2007-06-07 Thread Dov Feldstern
Stefan Schimanski wrote: Am 07.06.2007 um 22:29 schrieb José Matos: On Thursday 07 June 2007 14:19:41 Stefan Schimanski wrote: Order to apply: no_bidi_isboundary.patch no_bidi_setcurrentfont.patch local_bidi.patch cursor_get_font.patch rtl_spaces.patch What is missing to apply from this

Re: [patch] KeyMap bug (again)

2007-06-07 Thread Dov Feldstern
José Matos wrote: On Thursday 07 June 2007 08:47:24 Jean-Marc Lasgouttes wrote: The patch is OK to me (with the extra switchKeymap). Dov if you come with a patch with this suggestion implemented then you have my OK. :-) JMarc Thanks, I will send a corrected patch soon.

Re: Spaces on RTL boundaries

2007-06-07 Thread Stefan Schimanski
Am 07.06.2007 um 22:51 schrieb Dov Feldstern: Stefan Schimanski wrote: Am 07.06.2007 um 22:29 schrieb José Matos: On Thursday 07 June 2007 14:19:41 Stefan Schimanski wrote: Order to apply: no_bidi_isboundary.patch no_bidi_setcurrentfont.patch local_bidi.patch cursor_get_font.patch

compile error

2007-06-07 Thread Stefan Schimanski
drawMarkers moved to InsetMathNest?! See r18701. Please revert or fix that. /Users/sts/Quellen/mac/lyx-devel/src/mathed/InsetMathMBox.cpp: In member function 'virtual void lyx::InsetMathMBox::draw (lyx::PainterInfo&, int, int) const':

Re: [patch] KeyMap bug (again)

2007-06-07 Thread Dov Feldstern
Dov Feldstern wrote: José Matos wrote: On Thursday 07 June 2007 08:47:24 Jean-Marc Lasgouttes wrote: The patch is OK to me (with the extra switchKeymap). Dov if you come with a patch with this suggestion implemented then you have my OK. :-) JMarc Thanks, I will send a corrected

Re: compile error

2007-06-07 Thread Bo Peng
On 6/7/07, Stefan Schimanski <[EMAIL PROTECTED]> wrote: drawMarkers moved to InsetMathNest?! See r18701. Please revert or fix that. Interesting, why my compiler did not complain? (The test was also tested by others). I will see what is going on... Bo

Re: EndEnvironment layout entry.

2007-06-07 Thread Richard Heck
Bo Peng wrote: I can polish this and add lyx2lyx. I guess it is better called 'Splitter' or 'Separator', and is displayed as '--- split environment ---' Othewise, we will see lyx files with all environment properly ended with this environment. Yes, that seems right. I had a similar idea. rh

Re: compile error

2007-06-07 Thread Bo Peng
On 6/7/07, Bo Peng <[EMAIL PROTECTED]> wrote: On 6/7/07, Stefan Schimanski <[EMAIL PROTECTED]> wrote: > drawMarkers moved to InsetMathNest?! See r18701. Please revert or fix > that. The patch is partially reverted to move drawMarkers and drawMarkers2 back to its original place. The problem

Re: [patch] KeyMap bug (again)

2007-06-07 Thread Jean-Marc Lasgouttes
> "Dov" == Dov Feldstern <[EMAIL PROTECTED]> writes: Dov> But what do you think about this suggestion: instead of adding Dov> switchKeyMap in the new place, and where I added it originally, Dov> just add it inside translateAndInsert() itself? (I haven't tried Dov> this out yet --- but if it

Re: [PATCH] 3782 Crash with selection of table cells

2007-06-07 Thread Jean-Marc Lasgouttes
> "Alfredo" == Alfredo Braunstein <[EMAIL PROTECTED]> writes: >> That makes sense but as I have said (or implied), to avoid >> introduction of new bugs, it is unwise to overhaul clipboards now. >> JMarc's two-liner stops crashes, my two-liners saves selection >> correctly for that particular

Re: [PATCH] 3782 Crash with selection of table cells

2007-06-07 Thread Bo Peng
Alfredo> Seems fair enough ;-) Go for it, Will do. but Alfredo's suggestion is the way to go later. I hope Alfredo will go further enough to merge text and table clipboards. :-) Bo

Re: compile error

2007-06-07 Thread José Matos
On Thursday 07 June 2007 22:06:47 Bo Peng wrote: > Interesting, why my compiler did not complain? (The test was also > tested by others). I had no problems with gcc 4.1.2 (with bits from 4.2) or else I would have complained earlier. :-) -- José Abílio

Re: EndEnvironment layout entry.

2007-06-07 Thread Jean-Marc Lasgouttes
> "José" == José Matos <[EMAIL PROTECTED]> writes: José> I have a personal agenda against this kind of hacks. :-) I José> know because I have used them in the past. I'd rather avoid such hacks too, since they do not add much we cannot already can do. José> The solution would be to go to

Re: Converter problem with Mac OS packages

2007-06-07 Thread José Matos
On Tuesday 05 June 2007 21:32:05 Mael Hilléreau wrote: > Le 5 juin 07 à 19:14, Andre Poenitz a écrit : > > Pleas send patches (also) to the mailing list unless they are > > exceptionally large. And please create 'unified diffs'. People > > here are used to them... > > Sorry about that, I updated

Re: EndEnvironment layout entry.

2007-06-07 Thread José Matos
On Thursday 07 June 2007 23:01:16 Jean-Marc Lasgouttes wrote: > The 'only' problem is a very tough one... > > Another solution is to remove this special handling for environments > and force to use depth when several paragraphs are in a same env. > > JMarc It needs carefull thought but it is not

Re: EndEnvironment layout entry.

2007-06-07 Thread Richard Heck
Jean-Marc Lasgouttes wrote: José> I have a personal agenda against this kind of hacks. :-) I José> know because I have used them in the past. I'd rather avoid such hacks too, since they do not add much we cannot already can do. José> The solution would be to go to a box based approach

Re: Converter problem with Mac OS packages

2007-06-07 Thread José Matos
On Tuesday 05 June 2007 22:11:14 Jean-Marc Lasgouttes wrote: > > He is just trying to please his new boss :) > > JMarc As it can be seen here in the second photo taken in Berlin (first person standing in the left side): http://labs.trolltech.com/blogs/2007/05/30/qt-430-released/ -- José

Road to rc2

2007-06-07 Thread José Matos
Hi, I have changed the proposed schedule for 1.5.0 taking into account the delay in the release of rc1. The new schedule can be found in the usual place: http://wiki.lyx.org/Devel/ReleaseSchedule (Executive summary: rc2 - 15 June; final - 22 June) I would like to only allow

Re: Road to rc2

2007-06-07 Thread Bennett Helm
On Jun 7, 2007, at 7:01 PM, José Matos wrote: Hi, I have changed the proposed schedule for 1.5.0 taking into account the delay in the release of rc1. The new schedule can be found in the usual place: http://wiki.lyx.org/Devel/ReleaseSchedule (Executive summary: rc2 - 15 June; final - 22

Re: Spaces on RTL boundaries

2007-06-07 Thread Dov Feldstern
Stefan Schimanski wrote: Guy in the other thread prefers some magic here that the first space is moved to the other side (so in fact the LyX 1.3 behavior. What do you think? We could probably implement such a magic in the insertChar handler. Right, and so does Ran. Attached is a patch

Re: Spaces on RTL boundaries

2007-06-07 Thread Dov Feldstern
Sorry, please use the corrected patch attached... Dov Feldstern wrote: Stefan Schimanski wrote: Guy in the other thread prefers some magic here that the first space is moved to the other side (so in fact the LyX 1.3 behavior. What do you think? We could probably implement such a magic in

Re: EndEnvironment layout entry.

2007-06-07 Thread hzluo
An environment paragraph would be inserted inside a box, allowing other things to be inserted before the begin of the first item.I like this idea a lot :-) Then we can customize the environment with ERT.Or else the first item will not be customized...

[PATCH] Allow parameters to bypass InsetListingsParams validation.

2007-06-07 Thread Bo Peng
The validation mechanism of InsetListingsParams is great but is not flexible to handle unrecognized parameters introduced, e.g., by a new version of the listings. The attached patch allows users to input arbitrary parameters by prefixing it with a '@' sign. @ will be removed in latex output, but

Re: compile error

2007-06-07 Thread Stefan Schimanski
Strange, maybe my checkout it broken. But as far as I could see with my tired eyes yesterday the mentioned patch moves drawMarkers from InsetMath to InsetMathNest. And as InsetMathMBox is derived from the former there should be an error, no?! Stefan On Thursday 07 June 2007 22:06:47 Bo

Re: Road to rc2

2007-06-07 Thread Stefan Schimanski
Am 08.06.2007 um 01:22 schrieb Bennett Helm: On Jun 7, 2007, at 7:01 PM, José Matos wrote: Hi, I have changed the proposed schedule for 1.5.0 taking into account the delay in the release of rc1. The new schedule can be found in the usual place: http://wiki.lyx.org/Devel/ReleaseSchedule

Re: [PATCH] 2697: SplitLayout environment.

2007-06-07 Thread Martin Vermeer
On Thu, Jun 07, 2007 at 09:34:28PM -0500, Bo Peng wrote: > I can not believe that I am hitting a second WONTFIX in two days, but > this is likely the fate of this patch. > > The attached patch adds SplitLayout environment to a few layout/inc > files that will end up in almost all layouts. This

Re: Spaces on RTL boundaries

2007-06-07 Thread Andre Poenitz
On Thu, Jun 07, 2007 at 11:54:55AM +0200, Stefan Schimanski wrote: > Ran Rutenberg wrote: > >Hi, > > > >I tried to write a bidi file using the four patches and I didn't see > >a big difference in writing with the patches and without them. One > >small problem I did find is that when writing a word

Re: [PATCH-again] Bug 3821: Odd middle-click behavior

2007-06-07 Thread Andre Poenitz
On Thu, Jun 07, 2007 at 10:12:47AM -0400, Richard Heck wrote: > > Comments? OK? Make sense. Fine with me. Andre'

Re: [PATCH] 3810 --add check for evince

2007-06-07 Thread Andre Poenitz
On Fri, Jun 08, 2007 at 12:18:55AM +1000, Darren Freeman wrote: > On Thu, 2007-06-07 at 09:48 -0400, Richard Heck wrote: > > Is there some way to find out what the user has installed as his or her > > default pdf reader? Obviously, this will be highly platform sensitive, > > and even desktop

Re: [PATCH] 3810 --add check for evince

2007-06-07 Thread Andre Poenitz
On Fri, Jun 08, 2007 at 12:18:55AM +1000, Darren Freeman wrote: > On Thu, 2007-06-07 at 09:48 -0400, Richard Heck wrote: > > Is there some way to find out what the user has installed as his or her > > default pdf reader? Obviously, this will be highly platform sensitive, > > and even desktop

Re: [PATCH] 2697: SplitLayout environment.

2007-06-07 Thread Richard Heck
Martin Vermeer wrote: On Thu, Jun 07, 2007 at 09:34:28PM -0500, Bo Peng wrote: I need four positive votes to outnumber our future-oriented masters (who consider this as a useless ugly hack). I like it. And the future work needed to get to that "box-based" solution will be substantial.

Re: Road to rc2

2007-06-07 Thread Anders Ekberg
José Matos Thu, 07 Jun 2007 16:05:49 -0700 - What are the patches submitted that you think deserve attention? I would like to give special attention to critical bugs and so on. Since the most need for saving as 1.4.x should occur in the transition to 1.5, I think

Re: compile error

2007-06-07 Thread Andre Poenitz
On Thu, Jun 07, 2007 at 04:06:47PM -0500, Bo Peng wrote: > On 6/7/07, Stefan Schimanski <[EMAIL PROTECTED]> wrote: > >drawMarkers moved to InsetMathNest?! See r18701. Please revert or fix > >that. > > Interesting, why my compiler did not complain? (The test was also > tested by others). > > I

Re: compile error

2007-06-07 Thread Andre Poenitz
On Thu, Jun 07, 2007 at 04:36:56PM -0500, Bo Peng wrote: > On 6/7/07, Bo Peng <[EMAIL PROTECTED]> wrote: > >On 6/7/07, Stefan Schimanski <[EMAIL PROTECTED]> wrote: > >> drawMarkers moved to InsetMathNest?! See r18701. Please revert or fix > >> that. > > The patch is partially reverted to move

<    1   2   3