Re: [patch] Solving selection painting bugs

2008-09-13 Thread Pavel Sanda
Abdelrazak Younes wrote: > Both looks good to me. Somebody please commit them (or wait until you have > commit privilege). i did this on your request. pavel

please don't change any documentation file until bug 5250 is fixed

2008-09-13 Thread Uwe Stöhr
Currently any change on existing documents you make with LyX 1.6svn will delete table alignment informations: http://bugzilla.lyx.org/show_bug.cgi?id=5250 So please don't commit anything to documentation files until this is fixed. (I'm now off again for a week.) regards Uwe

Re: [blocker] toolbars are no longer displayed

2008-09-13 Thread Pavel Sanda
Uwe Stöhr wrote: > Uwe Stöhr schrieb: > >> I just made an SVN update and compiled LyX 1.6. But in contrary to Monday >> evening, no toolbar is shown. I can set up in the UI-files and the View >> menu settings what I want, but they won't be shown. >> -> a release blocker > > This bug is more worse

RE: [patch] Solving selection painting bugs

2008-09-13 Thread Vincent van Ravesteijn - TNW
>>mutable bool margin_beg_sel; >>mutable bool margin_end_sel; > >if these bools indicate whether you want to draw some selection to >either the left or right, why not simply call >them: > >leftmargin_sel >rightmargin_sel > >(just ignore me if it shows that i didn't study your patch) Yes,

RE: [patch] Solving selection painting bugs

2008-09-13 Thread leuven edwin
>mutable bool margin_beg_sel; >mutable bool margin_end_sel; if these bools indicate whether you want to draw some selection to either the left or right, why not simply call them: leftmargin_sel rightmargin_sel (just ignore me if it shows that i didn't study your patch)

Re: [blocker] toolbars are no longer displayed

2008-09-13 Thread Uwe Stöhr
Uwe Stöhr schrieb: I just made an SVN update and compiled LyX 1.6. But in contrary to Monday evening, no toolbar is shown. I can set up in the UI-files and the View menu settings what I want, but they won't be shown. -> a release blocker This bug is more worse: The whole toolbar handling is

Re: LyX currently uncompilable

2008-09-13 Thread Uwe Stöhr
Abdelrazak Younes schrieb: Upgrading boost would be the best thing to do... Who can do this? regards Uwe

Re: [patch] Solving selection painting bugs

2008-09-13 Thread Vincent van Ravesteijn
Abdelrazak Younes wrote: Yes, that sounds a better and cleaner solution. We basically only keep rows that are shown on screen in memory so adding two move variable is not a big problem. Abdel. Hi, I sort of cut the new code in four. It is no longer computed for and after the checksum and th

Re: Bug in CS 26372

2008-09-13 Thread Andre Poenitz
On Sat, Sep 13, 2008 at 05:21:27PM +0200, Abdelrazak Younes wrote: > Vincent van Ravesteijn - TNW wrote: >> >> >>> Vincent van Ravesteijn - TNW wrote: >>> In the following piece of code, the tmpInset was deleted if insetAllowed was false. After the change, tmpInset is deleted wh

Re: LyX currently uncompilable

2008-09-13 Thread Abdelrazak Younes
Uwe Stöhr wrote: One of todays changesets makes LyX uncompilable: I fixed that, an unfinished svn commit apparently, don't understand what happened. I'm btw. annoyed by this message: Unknown compiler version - please run the configure tests and report the results, LyX-TODO: update boost!

LyX currently uncompilable

2008-09-13 Thread Uwe Stöhr
One of todays changesets makes LyX uncompilable: Text2.cpp Unknown compiler version - please run the configure tests and report the results, LyX-TODO: update boost! D:\LyXSVN\lyx-devel\src\Text2.cpp(93) : error C2440: 'initializing' : cannot convert from 'const lyx::Inset' to 'const lyx::Inset

Re: [patch] Solving selection painting bugs

2008-09-13 Thread Abdelrazak Younes
Abdelrazak Younes wrote: Vincent van Ravesteijn - TNW wrote: As a general comment on the patch, I didn't review it in detail but I feel that this is a lot of additional code. The only new code is in the logic whether to draw the margins or not. And this was just the problem that I had with h

Re: [patch] Solving selection painting bugs

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn - TNW wrote: As a general comment on the patch, I didn't review it in detail but I feel that this is a lot of additional code. The only new code is in the logic whether to draw the margins or not. And this was just the problem that I had with how LyX drawn the

Re: Bug in CS 26372

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn - TNW wrote: Vincent van Ravesteijn - TNW wrote: In the following piece of code, the tmpInset was deleted if insetAllowed was false. After the change, tmpInset is deleted when insetAllowed is true. This leads to disappearing Insets and crashes. I'd suggest add

RE: Bug in CS 26372

2008-09-13 Thread Vincent van Ravesteijn - TNW
>Vincent van Ravesteijn - TNW wrote: >> In the following piece of code, the tmpInset was deleted if >> insetAllowed was false. After the change, tmpInset is deleted when >> insetAllowed is true. >> >> This leads to disappearing Insets and crashes. I'd suggest adding a !. >> > >Oups, sorry.

RE: [patch] Solving selection painting bugs

2008-09-13 Thread Vincent van Ravesteijn - TNW
> As a general comment on the patch, I didn't review it in detail but > I feel that this is a lot of additional code. The only new code is in the logic whether to draw the margins or not. And this was just the problem that I had with how LyX drawn the margins. Sometimes it does, sometimes with

Re: [patch] Solving selection painting bugs

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn wrote: Abdel, I took the liberty to change the drawParagraph function in TextMetrics.cpp a bit. This because, after you asked me to use meaningfull names for the variables, I tried to understand the other names in the function and I didn't succeed. I felt that variables

Re: [patch] Solving selection painting bugs

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn wrote: Vincent van Ravesteijn - TNW wrote: First, when only selecting the end_margin between 'after i-1' and 'before i', the anchor is incorrectly set to be equal to the cur. This is because comparing two CursorSlices does not take into account the boundary property (beca

Re: Missing package in (European) Lyx Standard Installation

2008-09-13 Thread sms
Uwe Stöhr schrieb: You need to have an opened Internet connection while installing LyX or while reconfiguring it. Then all packages needed by LyX are automatically installed. The alternative Windows installer provides the best possible support for this: http://wiki.lyx.org/Windows/LyXWinInstal

Re: [patch] Solving selection painting bugs

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn wrote: Abdel, I took the liberty to change the drawParagraph function in TextMetrics.cpp a bit. This because, after you asked me to use meaningfull names for the variables, I tried to understand the other names in the function and I didn't succeed. I felt that variables

Re: [blocker] toolbars are no longer displayed

2008-09-13 Thread Abdelrazak Younes
Uwe Stöhr wrote: > is that repaired when you unset > tools->preferences->Allow saving/restoring of windows geometry > save & restart lyx. Yes. But this option has always been by default turned on. And until Monday evening, this workaround wasn't needed. It cannot be that users first have to mo

Re: Bug in CS 26372

2008-09-13 Thread Abdelrazak Younes
Vincent van Ravesteijn - TNW wrote: In the following piece of code, the tmpInset was deleted if insetAllowed was false. After the change, tmpInset is deleted when insetAllowed is true. - if (!toPar.insetAllowed(tmpInset->lyxCode())) { + Inset * inset = toPar.inInset(); + if (inset&& inset->