Re: [Patch] Toc dialog cleanup

2007-03-12 Thread Abdelrazak Younes
José Matos wrote: On Tuesday 06 March 2007 5:49:50 pm Abdelrazak Younes wrote: Dear Jose, Here is the proposed patch. There's still one bug remaining in the type's list but it should be easy to solve. I propose to commit the safe part now (everything that is not in qt4) and, when the above bug

Re: [Patch] Toc dialog cleanup

2007-03-09 Thread Abdelrazak Younes
José Matos wrote: On Tuesday 06 March 2007 5:49:50 pm Abdelrazak Younes wrote: Dear Jose, Here is the proposed patch. There's still one bug remaining in the type's list but it should be easy to solve. I propose to commit the safe part now (everything that is not in qt4) and, when the above bug

Re: [Patch] Toc dialog cleanup

2007-03-09 Thread José Matos
On Tuesday 06 March 2007 5:49:50 pm Abdelrazak Younes wrote: > Dear Jose, > > Here is the proposed patch. There's still one bug remaining in the > type's list but it should be easy to solve. > > I propose to commit the safe part now (everything that is not in qt4) > and, when the above bug is solve

Re: [Patch] Toc dialog cleanup

2007-03-07 Thread Angus Leeming
Angus Leeming <[EMAIL PROTECTED]> writes: > > Another feature request: update the toc on buffer-switch instead of > > hiding it. > > Note that the infrastructure for this one is in place already. You just need > to tell frontends\Dialogs.C that this is an updateable dialog rather than a > hidea

Re: [Patch] Toc dialog cleanup

2007-03-07 Thread Angus Leeming
Abdelrazak Younes <[EMAIL PROTECTED]> writes: > Another feature request: update the toc on buffer-switch instead of > hiding it. Note that the infrastructure for this one is in place already. You just need to tell frontends\Dialogs.C that this is an updateable dialog rather than a hideable one.

[Patch] Toc dialog cleanup round 4

2007-03-07 Thread Abdelrazak Younes
Here is an updated patch that solve Leuven, E. wrote: it is very nice! a couple of comments (using the user guide): initially the level expansion of the tree does not match the slider this more or less initially the type combo is empty this when switching to type = table i can switch

Re: [Updated Patch] Toc dialog cleanup and a few other things

2007-03-07 Thread Abdelrazak Younes
Leuven, E. wrote: lyx crashes when openinc the toc with an empty doc Sorry, I forgot to include the attached change in the former patch... Abdel. Index: lyxfunc.C === --- lyxfunc.C (revision 17409) +++ lyxfunc.C (working copy)

RE: [Updated Patch] Toc dialog cleanup and a few other things

2007-03-07 Thread Leuven, E.
lyx crashes when openinc the toc with an empty doc -Original Message- From: news on behalf of Abdelrazak Younes Sent: Wed 3/7/07 16:26 To: lyx-devel@lists.lyx.org Subject: [Updated Patch] Toc dialog cleanup and a few other things Abdelrazak Younes wrote: > Dear Jose, > > He

[Updated Patch] Toc dialog cleanup and a few other things

2007-03-07 Thread Abdelrazak Younes
Abdelrazak Younes wrote: Dear Jose, Here is the proposed patch. There's still one bug remaining in the type's list but it should be easy to solve. I propose to commit the safe part now (everything that is not in qt4) and, when the above bug is solved, commit the rest. Here are the main dif

Re: [Patch] Toc dialog cleanup

2007-03-07 Thread Abdelrazak Younes
Leuven, E. wrote: it is very nice! I think so too :-) a couple of comments (using the user guide): initially the level expansion of the tree does not match the slider Known bug. initially the type combo is empty Known bug. when switching to type = table i can switch to table 1 and

RE: [Patch] Toc dialog cleanup

2007-03-06 Thread Leuven, E.
Younes Sent: Tue 3/6/07 18:49 To: lyx-devel@lists.lyx.org Subject: [Patch] Toc dialog cleanup Dear Jose, Here is the proposed patch. There's still one bug remaining in the type's list but it should be easy to solve. I propose to commit the safe part now (everything that is not in qt4)

[Patch] Toc dialog cleanup

2007-03-06 Thread Abdelrazak Younes
Dear Jose, Here is the proposed patch. There's still one bug remaining in the type's list but it should be easy to solve. I propose to commit the safe part now (everything that is not in qt4) and, when the above bug is solved, commit the rest. I won't work on this until tomorrow evening so,

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-06 Thread Abdelrazak Younes
Leuven, E. wrote: Excellent Edwin, you're a master! what are u then? :-) I know, I should RTFM but what is the trick? cast a magic spell i learned from harry potter himself. some - second rate wizards - have claimed that selecting the base widget (clicking on it) and then layout in a g

RE: Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-06 Thread Leuven, E.
> Excellent Edwin, you're a master! what are u then? > I know, I should RTFM but what is the trick? cast a magic spell i learned from harry potter himself. some - second rate wizards - have claimed that selecting the base widget (clicking on it) and then layout in a grid (ctrl+5) also works un

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-06 Thread Abdelrazak Younes
Leuven, E. wrote: something like the attached? Excellent Edwin, you're a master! I know, I should RTFM but what is the trick? Abdel.

RE: Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-06 Thread Leuven, E.
> Edwin! I knew I forgot someone in my call for help ;-) i am not much help these days, but i still manage to lurk ;-)

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-06 Thread Abdelrazak Younes
Leuven, E. wrote: Edwin! I knew I forgot someone in my call for help ;-) something like the attached? I'll try that, thanks. Abdel.

RE: Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-06 Thread Leuven, E.
something like the attached? -Original Message- From: news on behalf of Abdelrazak Younes Sent: Tue 3/6/07 16:11 To: lyx-devel@lists.lyx.org Subject: Re: [Patch] TOC Dialog crashes with empty toc list José Matos wrote: > On Monday 05 March 2007 6:27:26 pm Abdelrazak Younes wr

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Abdelrazak Younes
Bernhard Roider wrote: Abdelrazak Younes wrote: Bernhard Roider wrote: Hello, i found a crash when one of the buttons in the TOC dialog is pressed if the TOC is empty. The attached patch disables the buttons if there are no entries in the TOC dialog. To be safe it also adds a test to the bu

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Bernhard Roider
Abdelrazak Younes wrote: Bernhard Roider wrote: Hello, i found a crash when one of the buttons in the TOC dialog is pressed if the TOC is empty. The attached patch disables the buttons if there are no entries in the TOC dialog. To be safe it also adds a test to the button handler functions w

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread José Matos
On Monday 05 March 2007 6:27:26 pm Abdelrazak Younes wrote: > >   How much time do you need? > > A few days... Granted. :-) > >   I suggested the freeze with a very small window of commit time > > remaining. I am aware that it is not nice to change the rules in the > > middle of the game, that

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Abdelrazak Younes
José Matos wrote: On Monday 05 March 2007 4:47:31 pm Abdelrazak Younes wrote: Hello Bernhard, I don't know if you are planning to commit this one but I wanted to let you know that I integrated your changes already in my local tree. I am basically rewriting a good part of this code and I hope th

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread José Matos
On Monday 05 March 2007 4:47:31 pm Abdelrazak Younes wrote: > Hello Bernhard, > > I don't know if you are planning to commit this one but I wanted to let > you know that I integrated your changes already in my local tree. I am > basically rewriting a good part of this code and I hope that it will b

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Abdelrazak Younes
Bernhard Roider wrote: Hello, i found a crash when one of the buttons in the TOC dialog is pressed if the TOC is empty. The attached patch disables the buttons if there are no entries in the TOC dialog. To be safe it also adds a test to the button handler functions whether the list has entrie

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Abdelrazak Younes
Lars Gullik Bjønnes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: | > Abdelrazak Younes <[EMAIL PROTECTED]> writes: | > | Lars Gullik Bjønnes wrote: | > | > Bernhard Roider <[EMAIL PROTECTED]> writes: | > | | > | @@ -150,9 +153,12 @@ | > | > | void QTocDialog

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Lars Gullik Bjønnes
Abdelrazak Younes <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: | > Abdelrazak Younes <[EMAIL PROTECTED]> writes: | > | Lars Gullik Bjønnes wrote: | > | > Bernhard Roider <[EMAIL PROTECTED]> writes: | > | | > | @@ -150,9 +153,12 @@ | > | > | void QTocDialog::on_moveDownPB_clicked() | >

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Abdelrazak Younes
Lars Gullik Bjønnes wrote: Abdelrazak Younes <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: | > Bernhard Roider <[EMAIL PROTECTED]> writes: | | > | @@ -150,9 +153,12 @@ | > | void QTocDialog::on_moveDownPB_clicked() | > | { | > same comment | | Same comment to what? So you didn'

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-05 Thread Lars Gullik Bjønnes
Abdelrazak Younes <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: | > Bernhard Roider <[EMAIL PROTECTED]> writes: | | > | @@ -150,9 +153,12 @@ | > | void QTocDialog::on_moveDownPB_clicked() | > | { | > same comment | | Same comment to what? So you didn't read the patch did you? | Wh

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-04 Thread Abdelrazak Younes
Lars Gullik Bjønnes wrote: Bernhard Roider <[EMAIL PROTECTED]> writes: | @@ -150,9 +153,12 @@ | void QTocDialog::on_moveDownPB_clicked() | { same comment Same comment to what? | @@ -160,9 +166,12 @@ | void QTocDialog::on_moveInPB_clicked() | { same comment ditto | @@ -170,9 +17

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-04 Thread Bernhard Roider
Lars Gullik Bjønnes wrote: A space after '&' as well please. Also; can list be a const reference? it can be a const reference --> new patch attached - now really Does the enableButtons() have a default arg? If that is our own function I think it should be removed so that have to use true/

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-04 Thread Bernhard Roider
Lars Gullik Bjønnes wrote: A space after '&' as well please. Also; can list be a const reference? it can be a const reference --> new patch attached Does the enableButtons() have a default arg? If that is our own function I think it should be removed so that have to use true/false explict

Re: [Patch] TOC Dialog crashes with empty toc list

2007-03-03 Thread Lars Gullik Bjønnes
Bernhard Roider <[EMAIL PROTECTED]> writes: | Hello, | | i found a crash when one of the buttons in the TOC dialog is pressed | if the TOC is empty. The attached patch disables the buttons if there | are no entries in the TOC dialog. To be safe it also adds a test to | the button handler function

[Patch] TOC Dialog crashes with empty toc list

2007-03-03 Thread Bernhard Roider
Hello, i found a crash when one of the buttons in the TOC dialog is pressed if the TOC is empty. The attached patch disables the buttons if there are no entries in the TOC dialog. To be safe it also adds a test to the button handler functions whether the list has entries before accessing the on

Re: [patch] TOC dialog slider

2006-11-29 Thread Ozgur Ugras BARAN
I had a chance to have a look at the bug in Toc dialog. I guess I found the source of the error. When the updateGui is called, the function ControlToc::getCurrentTocItem() is called. In this routine, item is turned by kernel().buffer().tocBackend().item(type,ParConstIterator it); and ParConstI

Re: [patch] TOC dialog slider

2006-11-25 Thread Abdelrazak Younes
Bo Peng wrote: Good, then apply it ;-) The patch looks simple but I do not know that part of the code. Please evaluate the implementation, modify and apply it. It would be best if the 'click and move' problem Juergen and I noticed can be corrected. OK, I've taken the time to review it and to

Re: [patch] TOC dialog slider

2006-11-23 Thread Bo Peng
Good, then apply it ;-) The patch looks simple but I do not know that part of the code. Please evaluate the implementation, modify and apply it. It would be best if the 'click and move' problem Juergen and I noticed can be corrected. Cheers, Bo

Re: [patch] TOC dialog slider

2006-11-23 Thread Abdelrazak Younes
Juergen Spitzmueller wrote: sending this again, since it doesn't seem to get through first time. Sorry for potential double posting. Ozgur Ugras BARAN wrote: No problem Abdel and thanks Edwin.. To reduce confusion, I have attached the patch. I have tested the patch now, and it seems to fix t

Re: [patch] TOC dialog slider

2006-11-22 Thread Juergen Spitzmueller
Ozgur Ugras BARAN wrote: > No problem Abdel and thanks Edwin.. To reduce confusion, I have > attached the patch. I have tested the patch now, and it seems to fix the remaining issues (and yes, Abdel, this *is* an improvement ui-wise). The only remaining glitch I see is that the tree position is

Re: [patch] TOC dialog slider

2006-11-22 Thread Juergen Spitzmueller
sending this again, since it doesn't seem to get through first time. Sorry for potential double posting. Ozgur Ugras BARAN wrote: > No problem Abdel and thanks Edwin.. To reduce confusion, I have > attached the patch. I have tested the patch now, and it seems to fix the remaining issues (and ye

Re: [patch] TOC dialog slider

2006-11-22 Thread Bo Peng
Thanks, Bo. Juergen, can you also test it, to see if it is enough for your taste? At the end, you were the one who asked for the feature. I am not sure if this exists before the patch, but when I click on any item, it moves to the end of the dialog. This is very distracting since I have to re-re

Re: [patch] TOC dialog slider

2006-11-22 Thread Juergen Spitzmueller
Abdelrazak Younes wrote: > Only the current top level node is opened. The lower lower level child > nodes are kept closed. You can navigate (including collapse/open) quite > easily using the arrow keys. I'd say that this is quicker that to use > the mouse on the slider. How? It can't seem to make

Re: [patch] TOC dialog slider

2006-11-22 Thread Juergen Spitzmueller
Ozgur Ugras BARAN wrote: > Thanks, Bo. Juergen, can you also test it, to see if it is enough for > your taste? At the end, you were the one who asked for the feature. Yes. However, probably not before next week. Jürgen

Re: [patch] TOC dialog slider

2006-11-22 Thread Ozgur Ugras BARAN
Abdel, IMO, the last patch makes the TOC slider very useful. Otherwise, TOC collapses everytime you change the section, which is very annoying. On 11/22/06, Bo Peng <[EMAIL PROTECTED]> wrote: > Sorry Ugras, but I don't have much time today to test your patch. I > suggest that you send again the

Re: [patch] TOC dialog slider

2006-11-22 Thread Bo Peng
Sorry Ugras, but I don't have much time today to test your patch. I suggest that you send again the full patch. Then, maybe someone else can test and commit for you (Edwin, Bo, Georg?). If not, I'll try to do that at the end of the week. I heard my name being called. OK, I will try this patch du

Re: [patch] TOC dialog slider

2006-11-22 Thread Abdelrazak Younes
Juergen Spitzmueller wrote: Abdelrazak Younes wrote: The more I think about it, the more I think this slider is not useful for any purpose in the Toc dialog. In the former qt3 dialog, the slider was useful because the nodes won't collapse/open automatically when you navigate within the document.

Re: [patch] TOC dialog slider

2006-11-22 Thread Juergen Spitzmueller
Abdelrazak Younes wrote: > The more I think about it, the more I think this slider is not useful > for any purpose in the Toc dialog. In the former qt3 dialog, the slider > was useful because the nodes won't collapse/open automatically when you > navigate within the document. This is not the case w

Re: [patch] TOC dialog slider

2006-11-18 Thread Abdelrazak Younes
Abdelrazak Younes wrote: Ozgur Ugras BARAN wrote: On 11/17/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: Ozgur Ugras BARAN wrote: > attached patch implement slider in TOC dialog. > > I have implemented a getDepth function to get the model indices. In my > opinion, somebody should derive a ne

Re: [patch] TOC dialog slider

2006-11-18 Thread Abdelrazak Younes
Ozgur Ugras BARAN wrote: On 11/17/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: Ozgur Ugras BARAN wrote: > attached patch implement slider in TOC dialog. > > I have implemented a getDepth function to get the model indices. In my > opinion, somebody should derive a new qvariant class for tocMo

Re: [patch] TOC dialog slider

2006-11-18 Thread Ozgur Ugras BARAN
On 11/17/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: Ozgur Ugras BARAN wrote: > attached patch implement slider in TOC dialog. > > I have implemented a getDepth function to get the model indices. In my > opinion, somebody should derive a new qvariant class for tocModelItem > which saves the

Re: [patch] TOC dialog slider

2006-11-17 Thread Abdelrazak Younes
Ozgur Ugras BARAN wrote: attached patch implement slider in TOC dialog. I have implemented a getDepth function to get the model indices. In my opinion, somebody should derive a new qvariant class for tocModelItem which saves the string data and depth information. that will save the depth calcula

Re: [patch] TOC dialog

2006-11-17 Thread Ozgur Ugras BARAN
On 11/17/06, Jean-Marc Lasgouttes <[EMAIL PROTECTED]> wrote: > "Ozgur" == Ozgur Ugras BARAN <[EMAIL PROTECTED]> writes: Ozgur> In short, there is something weird in cursor control for table Ozgur> floats. (other floats does not have the same problem). Is it with table floats themselves, or

Re: [patch] TOC dialog slider

2006-11-17 Thread Ozgur Ugras BARAN
On 11/17/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: Ozgur Ugras BARAN wrote: > attached patch implement slider in TOC dialog. > > I have implemented a getDepth function to get the model indices. In my > opinion, somebody should derive a new qvariant class for tocModelItem > which saves the

Re: [patch] TOC dialog

2006-11-17 Thread Jean-Marc Lasgouttes
> "Ozgur" == Ozgur Ugras BARAN <[EMAIL PROTECTED]> writes: Ozgur> In short, there is something weird in cursor control for table Ozgur> floats. (other floats does not have the same problem). Is it with table floats themselves, or table insets? JMarc

[patch] TOC dialog

2006-11-17 Thread Ozgur Ugras BARAN
On 11/16/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: Ozgur Ugras BARAN wrote: > Following patch corrects some misbehavior in TOC dialog: > > - When clicking on Toc insets, dialog switches to correct list. > - cursor goes to correct entry when clicked.. > - some other fixes.. I have tested a

Re: [patch] TOC dialog slider

2006-11-17 Thread Abdelrazak Younes
Ozgur Ugras BARAN wrote: attached patch implement slider in TOC dialog. I have implemented a getDepth function to get the model indices. In my opinion, somebody should derive a new qvariant class for tocModelItem which saves the string data and depth information. that will save the depth calcula

Re: [patch] TOC dialog

2006-11-17 Thread younes . a
Quoting Ozgur Ugras BARAN <[EMAIL PROTECTED]>: > On 11/16/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: > > Ozgur Ugras BARAN wrote: > > > Following patch corrects some misbehavior in TOC dialog: > > > > > > - When clicking on Toc insets, dialog switches to correct list. > > > - cursor goes to

[patch] TOC dialog slider

2006-11-17 Thread Ozgur Ugras BARAN
attached patch implement slider in TOC dialog. I have implemented a getDepth function to get the model indices. In my opinion, somebody should derive a new qvariant class for tocModelItem which saves the string data and depth information. that will save the depth calculation. I'll be glad somebo

Re: [patch] TOC dialog

2006-11-16 Thread Ozgur Ugras BARAN
Thanks, Abdel.. Ugras On 11/16/06, Abdelrazak Younes <[EMAIL PROTECTED]> wrote: Ozgur Ugras BARAN wrote: > Following patch corrects some misbehavior in TOC dialog: > > - When clicking on Toc insets, dialog switches to correct list. > - cursor goes to correct entry when clicked.. > - some other

Re: [patch] TOC dialog

2006-11-16 Thread Abdelrazak Younes
Ozgur Ugras BARAN wrote: Following patch corrects some misbehavior in TOC dialog: - When clicking on Toc insets, dialog switches to correct list. - cursor goes to correct entry when clicked.. - some other fixes.. I have tested and committed the patch. Thanks Ugras. There is one strange prob

Re: [patch] TOC dialog

2006-11-15 Thread Abdelrazak Younes
Ozgur Ugras BARAN wrote: Following patch corrects some misbehavior in TOC dialog: - When clicking on Toc insets, dialog switches to correct list. - cursor goes to correct entry when clicked.. - some other fixes.. This looks good at first glance. There is one strange problem, though. After c

[patch] TOC dialog

2006-11-15 Thread Ozgur Ugras BARAN
Following patch corrects some misbehavior in TOC dialog: - When clicking on Toc insets, dialog switches to correct list. - cursor goes to correct entry when clicked.. - some other fixes.. There is one strange problem, though. After clicking table toc entries, cursor goes to correct table, but th