Re: Various change tracking issues

2006-01-16 Thread Juergen Spitzmueller
Juergen Spitzmueller wrote: May I have the row-painter fix as a separate patch, please? Attached are the two fixes in separate patches. Comments? Jürgen

Re: Various change tracking issues

2006-01-16 Thread Lars Gullik Bjønnes
Juergen Spitzmueller [EMAIL PROTECTED] writes: | Juergen Spitzmueller wrote: |May I have the row-painter fix as a separate patch, please? | Attached are the two fixes in separate patches. | | Comments? I'd like Martin to test patch one, and commit that if it seems ok. Ad. patch2 I'd like

Re: Various change tracking issues

2006-01-16 Thread Martin Vermeer
On Mon, Jan 16, 2006 at 01:25:36PM +0100, Lars Gullik Bjønnes wrote: Juergen Spitzmueller [EMAIL PROTECTED] writes: | Juergen Spitzmueller wrote: |May I have the row-painter fix as a separate patch, please? | Attached are the two fixes in separate patches. | | Comments? I'd like

Re: Various change tracking issues

2006-01-16 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: I'd like Martin to test patch one, and commit that if it seems ok. I'll commit it in a moment. Ad. patch2 I'd like to wait for 1.4.1 with that one. OK. Jürgen

Re: Various change tracking issues

2006-01-16 Thread Juergen Spitzmueller
Juergen Spitzmueller wrote: > > > May I have the row-painter fix as a separate patch, please? > Attached are the two fixes in separate patches. Comments? Jürgen

Re: Various change tracking issues

2006-01-16 Thread Lars Gullik Bjønnes
Juergen Spitzmueller <[EMAIL PROTECTED]> writes: | Juergen Spitzmueller wrote: | > > > May I have the row-painter fix as a separate patch, please? | > Attached are the two fixes in separate patches. | | Comments? I'd like Martin to test patch one, and commit that if it seems ok. Ad. patch2 I'd

Re: Various change tracking issues

2006-01-16 Thread Martin Vermeer
On Mon, Jan 16, 2006 at 01:25:36PM +0100, Lars Gullik Bjønnes wrote: > Juergen Spitzmueller <[EMAIL PROTECTED]> writes: > > | Juergen Spitzmueller wrote: > | > > > May I have the row-painter fix as a separate patch, please? > | > Attached are the two fixes in separate patches. > | > | Comments?

Re: Various change tracking issues

2006-01-16 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: > I'd like Martin to test patch one, and commit that if it seems ok. I'll commit it in a moment. > Ad. patch2 I'd like to wait for 1.4.1 with that one. OK. Jürgen

Re: Various change tracking issues

2006-01-15 Thread Bo Peng
Can anyone confirm http://bugzilla.lyx.org/show_bug.cgi?id=2214 ? I have checked that dvipost can overstrike formula and this is a latex-generating problem of lyx. Bo

Re: Various change tracking issues

2006-01-15 Thread Bo Peng
Can anyone confirm http://bugzilla.lyx.org/show_bug.cgi?id=2214 ? I have checked that dvipost can overstrike formula and this is a latex-generating problem of lyx. Bo

Re: Various change tracking issues

2006-01-12 Thread Juergen Spitzmueller
Juergen Spitzmueller wrote: Lars Gullik Bjønnes wrote: It changes some details that I am not sure will not have side-effects. I cannot exclude that in last consequence, of course, even if I've tested lots of possible cases. Maybe someone else could give it a hard test. | I think this

Re: Various change tracking issues

2006-01-12 Thread Juergen Spitzmueller
Juergen Spitzmueller wrote: > Lars Gullik Bjønnes wrote: > > It changes some details that I am not sure will not have side-effects. > > I cannot exclude that in last consequence, of course, even if I've tested > lots of possible cases. Maybe someone else could give it a hard test. > > > | > I

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: I am not sure whether or not this has been discussed, but I would like to add PASTED TEXT IS NOT CONSIDERED AS CHANGE 1. new doc 2. write something, copy 3. turn on change tracking 4. paste Very strange. I have added a bugzilla report... Michael

Re: Various change tracking issues

2006-01-11 Thread Bo Peng
PASTED TEXT IS NOT CONSIDERED AS CHANGE Very strange. I have added a bugzilla report... This is not very strange in that the *not-changed property' of copied lyx text is somehow preserved. If I paste from outside of lyx using my middle button, the added text is considered as change. Bo

Re: Various change tracking issues

2006-01-11 Thread Bo Peng
Another change tracking bug: CAN NOT ACCEPT THE FIRST CHANGE (?) 1. open a lyx document 2. type in a, newline, b, newline 3. turn on change tracking 4. add change 1 after line b 5, add change 2 after line a ( get a change 2 b change 1 ) 6. Go to the beginning of the

Re: Various change tracking issues

2006-01-11 Thread Bo Peng
Another one (hope it has not been reported) CHANGE OF FORMULA IS NOT TRACED 1. open a lyx document 2. insert $x^2$ 3. turn on change tracking 4. go into the formula and remove ^2 5. $x$ is not marked as changed. Shall we advertise change-tracking in 1.4.0 as experimental? I understand 1.4.0

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: Another one (hope it has not been reported) CHANGE OF FORMULA IS NOT TRACED 1. open a lyx document 2. insert $x^2$ 3. turn on change tracking 4. go into the formula and remove ^2 5. $x$ is not marked as changed. Added to bugzilla. Shall we advertise change-tracking in

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: CAN NOT ACCEPT THE FIRST CHANGE (?) 1. open a lyx document 2. type in a, newline, b, newline 3. turn on change tracking 4. add change 1 after line b 5, add change 2 after line a ( get a change 2 b change 1 ) 6. Go to the beginning of the document, accept change, lyx

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: Could you please add it by yourself (http://bugzilla.lyx.org)? I feel a little bit tired... :-) Added, but I could not find your bug reports (in list all unconfirmed bugs). Check for NEW bugs (I confirmed them). Or check for the latest bugs. Or for component

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: I am not sure whether or not this has been discussed, but I would like to add PASTED TEXT IS NOT CONSIDERED AS CHANGE 1. new doc 2. write something, copy 3. turn on change tracking 4. paste Very strange. I have added a bugzilla report... Michael

Re: Various change tracking issues

2006-01-11 Thread Bo Peng
> >PASTED TEXT IS NOT CONSIDERED AS CHANGE > > > Very strange. I have added a bugzilla report... This is not very strange in that the *not-changed property' of copied lyx text is somehow preserved. If I paste from outside of lyx using my middle button, the added text is considered as change. Bo

Re: Various change tracking issues

2006-01-11 Thread Bo Peng
Another change tracking bug: CAN NOT ACCEPT THE FIRST CHANGE (?) 1. open a lyx document 2. type in a, newline, b, newline 3. turn on change tracking 4. add change 1 after line b 5, add change 2 after line a ( get a change 2 b change 1 ) 6. Go to the beginning of the

Re: Various change tracking issues

2006-01-11 Thread Bo Peng
Another one (hope it has not been reported) CHANGE OF FORMULA IS NOT TRACED 1. open a lyx document 2. insert $x^2$ 3. turn on change tracking 4. go into the formula and remove ^2 5. $x$ is not marked as changed. Shall we advertise change-tracking in 1.4.0 as experimental? I understand 1.4.0

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: Another one (hope it has not been reported) CHANGE OF FORMULA IS NOT TRACED 1. open a lyx document 2. insert $x^2$ 3. turn on change tracking 4. go into the formula and remove ^2 5. $x$ is not marked as changed. Added to bugzilla. Shall we advertise change-tracking in

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: CAN NOT ACCEPT THE FIRST CHANGE (?) 1. open a lyx document 2. type in a, newline, b, newline 3. turn on change tracking 4. add change 1 after line b 5, add change 2 after line a ( get a change 2 b change 1 ) 6. Go to the beginning of the document, accept change, lyx

Re: Various change tracking issues

2006-01-11 Thread Michael Gerz
Bo Peng wrote: Could you please add it by yourself (http://bugzilla.lyx.org)? I feel a little bit tired... :-) Added, but I could not find your bug reports (in list all unconfirmed bugs). Check for "NEW" bugs (I confirmed them). Or check for the latest bugs. Or for component

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Jean-Marc Lasgouttes wrote: Juergen It is as simple as the attached. I've tested pretty Juergen intensive, some further testing would be welcome, though. What does the rowpainter change do? Martin's latest rowpainter change assures that rows are always repainted because of change tracking.

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: I must admit that I find the patch not simple enough :-) Sure? I find it pretty straightforward. I think this should wait so that it can get some more testing first. Your call of course. Jürgen

Re: Various change tracking issues

2006-01-10 Thread Lars Gullik Bjønnes
Juergen Spitzmueller [EMAIL PROTECTED] writes: | Lars Gullik Bjønnes wrote: | | I must admit that I find the patch not simple enough :-) | | Sure? I find it pretty straightforward. It changes some details that I am not sure will not have side-effects. | I think this should wait so that it

Re: Various change tracking issues

2006-01-10 Thread Martin Vermeer
On Tue, 2006-01-10 at 10:05 +0100, Juergen Spitzmueller wrote: Jean-Marc Lasgouttes wrote: Juergen It is as simple as the attached. I've tested pretty Juergen intensive, some further testing would be welcome, though. What does the rowpainter change do? Martin's latest rowpainter

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Martin Vermeer wrote: Now I am not sure anymore. Please test your patch also without the rowpainter change and see if it works. endpos is *one past the end*, i.e., the cursor is on the right side of all of the row's characters. It shouldn't do any change tracking marking there. It doesn't

Re: Various change tracking issues

2006-01-10 Thread Martin Vermeer
On Tue, Jan 10, 2006 at 12:13:31PM +0100, Juergen Spitzmueller wrote: Martin Vermeer wrote: Now I am not sure anymore. Please test your patch also without the rowpainter change and see if it works. endpos is *one past the end*, i.e., the cursor is on the right side of all of the row's

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: It changes some details that I am not sure will not have side-effects. I cannot exclude that in last consequence, of course, even if I've tested lots of possible cases. Maybe someone else could give it a hard test. | I think this should wait so that it can get some

Re: Various change tracking issues

2006-01-10 Thread Michael Gerz
Juergen Spitzmueller wrote: It is as simple as the attached. I've tested pretty intensive, some further testing would be welcome, though. Works like a charm! Lars? Please also note my new bug reports #2202 and #2203. The first one is a bit critical because LyX crashes (but this has

Re: Various change tracking issues

2006-01-10 Thread Bo Peng
I have produced a set of test cases. I thought it might be better to send them to the mailing list before adding a couple of bugzilla entries. I am not sure whether or not this has been discussed, but I would like to add PASTED TEXT IS NOT CONSIDERED AS CHANGE 1. new doc 2. write something,

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Jean-Marc Lasgouttes wrote: > Juergen> It is as simple as the attached. I've tested pretty > Juergen> intensive, some further testing would be welcome, though. > > What does the rowpainter change do? Martin's latest rowpainter change assures that rows are always repainted because of change

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: > I must admit that I find the patch not simple enough :-) Sure? I find it pretty straightforward. > I think this should wait so that it can get some more testing first. Your call of course. Jürgen

Re: Various change tracking issues

2006-01-10 Thread Lars Gullik Bjønnes
Juergen Spitzmueller <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: | | > I must admit that I find the patch not simple enough :-) | | Sure? I find it pretty straightforward. It changes some details that I am not sure will not have side-effects. | > I think this should wait so that

Re: Various change tracking issues

2006-01-10 Thread Martin Vermeer
On Tue, 2006-01-10 at 10:05 +0100, Juergen Spitzmueller wrote: > Jean-Marc Lasgouttes wrote: > > > Juergen> It is as simple as the attached. I've tested pretty > > Juergen> intensive, some further testing would be welcome, though. > > > > What does the rowpainter change do? > > Martin's latest

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Martin Vermeer wrote: > Now I am not sure anymore. Please test your patch also without the > rowpainter change and see if it works. endpos is *one past the end*, > i.e., the cursor is on the right side of all of the row's characters. It > shouldn't do any change tracking marking there. It

Re: Various change tracking issues

2006-01-10 Thread Martin Vermeer
On Tue, Jan 10, 2006 at 12:13:31PM +0100, Juergen Spitzmueller wrote: > Martin Vermeer wrote: > > > Now I am not sure anymore. Please test your patch also without the > > rowpainter change and see if it works. endpos is *one past the end*, > > i.e., the cursor is on the right side of all of the

Re: Various change tracking issues

2006-01-10 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: > It changes some details that I am not sure will not have side-effects. I cannot exclude that in last consequence, of course, even if I've tested lots of possible cases. Maybe someone else could give it a hard test. > | > I think this should wait so that it can get

Re: Various change tracking issues

2006-01-10 Thread Michael Gerz
Juergen Spitzmueller wrote: It is as simple as the attached. I've tested pretty intensive, some further testing would be welcome, though. Works like a charm! Lars? Please also note my new bug reports #2202 and #2203. The first one is a bit critical because LyX crashes (but this has

Re: Various change tracking issues

2006-01-10 Thread Bo Peng
> I have produced a set of test cases. I thought it might be better to > send them to the mailing list before adding a couple of bugzilla entries. I am not sure whether or not this has been discussed, but I would like to add PASTED TEXT IS NOT CONSIDERED AS CHANGE 1. new doc 2. write something,

Re: Various change tracking issues

2006-01-09 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER If this has a simple fix it should be done, otherwise it should wait. It is as simple as the attached. I've tested pretty intensive, some further testing would be welcome, though. Jürgen Index:

Re: Various change tracking issues

2006-01-09 Thread Jean-Marc Lasgouttes
Juergen == Juergen Spitzmueller [EMAIL PROTECTED] writes: Juergen Lars Gullik Bjønnes wrote: | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER If this has a simple fix it should be done, otherwise it should wait. Juergen It is as simple as the attached. I've tested pretty Juergen

Re: Various change tracking issues

2006-01-09 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes [EMAIL PROTECTED] writes: | Juergen == Juergen Spitzmueller [EMAIL PROTECTED] writes: | | Juergen Lars Gullik Bjønnes wrote: | | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER | | If this has a simple fix it should be done, otherwise it should | wait. | |

Re: Various change tracking issues

2006-01-09 Thread Martin Vermeer
On Mon, Jan 09, 2006 at 04:51:41PM +0100, Jean-Marc Lasgouttes wrote: Juergen == Juergen Spitzmueller [EMAIL PROTECTED] writes: Juergen Lars Gullik Bjønnes wrote: | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER If this has a simple fix it should be done, otherwise it should

Re: Various change tracking issues

2006-01-09 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: > | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER > > If this has a simple fix it should be done, otherwise it should wait. It is as simple as the attached. I've tested pretty intensive, some further testing would be welcome, though. Jürgen Index:

Re: Various change tracking issues

2006-01-09 Thread Jean-Marc Lasgouttes
> "Juergen" == Juergen Spitzmueller <[EMAIL PROTECTED]> writes: Juergen> Lars Gullik Bjønnes wrote: >> | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER >> >> If this has a simple fix it should be done, otherwise it should >> wait. Juergen> It is as simple as the attached. I've

Re: Various change tracking issues

2006-01-09 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | > "Juergen" == Juergen Spitzmueller <[EMAIL PROTECTED]> writes: | | Juergen> Lars Gullik Bjønnes wrote: | >> | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER | >> | >> If this has a simple fix it should be done, otherwise it

Re: Various change tracking issues

2006-01-09 Thread Martin Vermeer
On Mon, Jan 09, 2006 at 04:51:41PM +0100, Jean-Marc Lasgouttes wrote: > > "Juergen" == Juergen Spitzmueller <[EMAIL PROTECTED]> writes: > > Juergen> Lars Gullik Bjønnes wrote: > >> | DELETE KEY DOES NOT MOVE THE CURSOR TO THE NEXT CHARACTER > >> > >> If this has a simple fix it should be

Re: Various change tracking issues

2006-01-02 Thread Martin Vermeer
On Mon, Jan 02, 2006 at 01:10:00AM +0100, Jean-Marc Lasgouttes wrote: | What is this a response to? Your 'let's always repaint the row with the cursor in it' patch I belive. Right. My palm is not very good at quoting messages, sorry. JMarc -- Lgb For some reason mutt

Re: Various change tracking issues

2006-01-02 Thread Martin Vermeer
On Mon, Jan 02, 2006 at 01:10:00AM +0100, Jean-Marc Lasgouttes wrote: > | What is this a response to? > > >Your 'let's always repaint the row >with the cursor in it' patch I > >belive. > > Right. My palm is not very good at > quoting messages, sorry. > > JMarc > > -- > Lgb For some

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 03:25:31AM +0100, Lars Gullik Bjønnes wrote: Martin Vermeer [EMAIL PROTECTED] writes: ... | I'll commit the isChanged version if no-one objects. Verified that it | does the job for both deletions and insertions. Ok, with me. Actually I'd prefer to commit the

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Michael Gerz wrote: Juergen Spitzmueller wrote: My take on this is as follows: Fix the regressions that have been introduced by the rowSignature patch and leave all other things as they are. If we start to rewrite the change tracker code now (and things like paragraph split/merge will need

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Martin Vermeer wrote: Actually I'd prefer to commit the attached. It is more general in that it *always* repaints the row that the cursor is on, whether directly, or in some inset. So I modified the cursor position test routine, achieving a simplification too. Tested and appears to work as

Re: Various change tracking issues

2006-01-01 Thread lasgouttes
I cannot test it but it looks much better. I'd say 'apply it'. JMarc

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 02:00:58PM +0100, Michael Gerz wrote: Michael Gerz wrote: Juergen Spitzmueller wrote: My take on this is as follows: Fix the regressions that have been introduced by the rowSignature patch and leave all other things as they are. If we start to rewrite the change

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 02:00:58PM +0100, Michael Gerz wrote: Michael Gerz wrote: ... More investigation on paragraph breaks: Does not work: Pressing Enter in the middle of an unchanged line Works: Selecting copying paragraph break, pasting it in the middle of an unchanged line

Re: Various change tracking issues

2006-01-01 Thread Lars Gullik Bjønnes
Martin Vermeer [EMAIL PROTECTED] writes: | On Sun, Jan 01, 2006 at 02:00:58PM +0100, Michael Gerz wrote: | Michael Gerz wrote: | | ... | | | More investigation on paragraph breaks: | | Does not work: Pressing Enter in the middle of an unchanged line | Works: Selecting copying

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Martin Vermeer wrote: The code is here (text.C): 1026 void LyXText::breakParagraph(LCursor cur, bool keep_layout) 1027 { 1028 BOOST_ASSERT(this == cur.text()); 1029 // allow only if at start or end, or all previous is new text 1030 Paragraph cpar =

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 03:40:34PM +0100, Michael Gerz wrote: Martin Vermeer wrote: The code is here (text.C): 1026 void LyXText::breakParagraph(LCursor cur, bool keep_layout) 1027 { 1028 BOOST_ASSERT(this == cur.text()); 1029 // allow only if at start or end, or

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Martin Vermeer wrote: If inserting newlines directly _were_ allowed, they would not be rolled back together with the tracked changes to the surrounding characters. In that sense change tracking would be inconsistent. Cutting and pasting selections subvert this scheme. In other words: If

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 04:50:42PM +0100, Michael Gerz wrote: Martin Vermeer wrote: If inserting newlines directly _were_ allowed, they would not be rolled back together with the tracked changes to the surrounding characters. In that sense change tracking would be inconsistent. Cutting

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 02:17:00PM +0100, [EMAIL PROTECTED] wrote: I cannot test it but it looks much better. I'd say 'apply it'. JMarc What is this a response to? - Martin pgpHQabjkkkCG.pgp Description: PGP signature

Re: Various change tracking issues

2006-01-01 Thread Lars Gullik Bjønnes
Martin Vermeer [EMAIL PROTECTED] writes: | On Sun, Jan 01, 2006 at 02:17:00PM +0100, [EMAIL PROTECTED] wrote: | I cannot test it but it looks much | better. I'd say 'apply it'. | | JMarc | | What is this a response to? Your 'let's always repaint the row with the cursor in it' patch I

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 08:29:23PM +0100, Lars Gullik Bjønnes wrote: Martin Vermeer [EMAIL PROTECTED] writes: | On Sun, Jan 01, 2006 at 02:17:00PM +0100, [EMAIL PROTECTED] wrote: | I cannot test it but it looks much | better. I'd say 'apply it'. | | JMarc | | What is this a

Re: Various change tracking issues

2006-01-01 Thread Jean-Marc Lasgouttes
| What is this a response to? Your 'let's always repaint the row with the cursor in it' patch I belive. Right. My palm is not very good at quoting messages, sorry. JMarc -- Lgb

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 03:25:31AM +0100, Lars Gullik Bjønnes wrote: > Martin Vermeer <[EMAIL PROTECTED]> writes: ... > | I'll commit the isChanged version if no-one objects. Verified that it > | does the job for both deletions and insertions. > > Ok, with me. Actually I'd prefer to commit

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Michael Gerz wrote: Juergen Spitzmueller wrote: My take on this is as follows: Fix the regressions that have been introduced by the rowSignature patch and leave all other things as they are. If we start to rewrite the change tracker code now (and things like paragraph split/merge will need

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Martin Vermeer wrote: Actually I'd prefer to commit the attached. It is more general in that it *always* repaints the row that the cursor is on, whether directly, or in some inset. So I modified the cursor position test routine, achieving a simplification too. Tested and appears to work as

Re: Various change tracking issues

2006-01-01 Thread lasgouttes
I cannot test it but it looks much better. I'd say 'apply it'. JMarc

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 02:00:58PM +0100, Michael Gerz wrote: > Michael Gerz wrote: > > >Juergen Spitzmueller wrote: > > > >>My take on this is as follows: Fix the regressions that have been > >>introduced by the rowSignature patch and leave all other things as > >>they are. If we start to

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 02:00:58PM +0100, Michael Gerz wrote: > Michael Gerz wrote: ... > More investigation on paragraph breaks: > > Does not work: Pressing Enter in the middle of an unchanged line > Works: Selecting & copying paragraph break, pasting it in the middle > of an unchanged

Re: Various change tracking issues

2006-01-01 Thread Lars Gullik Bjønnes
Martin Vermeer <[EMAIL PROTECTED]> writes: | On Sun, Jan 01, 2006 at 02:00:58PM +0100, Michael Gerz wrote: | > Michael Gerz wrote: | | ... | | | > More investigation on paragraph breaks: | > | > Does not work: Pressing Enter in the middle of an unchanged line | > Works: Selecting &

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Martin Vermeer wrote: The code is here (text.C): 1026 void LyXText::breakParagraph(LCursor & cur, bool keep_layout) 1027 { 1028 BOOST_ASSERT(this == cur.text()); 1029 // allow only if at start or end, or all previous is new text 1030 Paragraph & cpar =

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 03:40:34PM +0100, Michael Gerz wrote: > Martin Vermeer wrote: > > >The code is here (text.C): > > > > 1026 void LyXText::breakParagraph(LCursor & cur, bool keep_layout) > > 1027 { > > 1028 BOOST_ASSERT(this == cur.text()); > > 1029 // allow only if at

Re: Various change tracking issues

2006-01-01 Thread Michael Gerz
Martin Vermeer wrote: If inserting newlines directly _were_ allowed, they would not be rolled back together with the tracked changes to the surrounding characters. In that sense change tracking would be inconsistent. Cutting and pasting selections subvert this scheme. In other words: If

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 04:50:42PM +0100, Michael Gerz wrote: > Martin Vermeer wrote: > > >If inserting newlines directly _were_ allowed, they would not be rolled > >back together with the tracked changes to the surrounding characters. In > >that sense change tracking would be inconsistent. > > >

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 02:17:00PM +0100, [EMAIL PROTECTED] wrote: > I cannot test it but it looks much > better. I'd say 'apply it'. > > JMarc What is this a response to? - Martin pgpHQabjkkkCG.pgp Description: PGP signature

Re: Various change tracking issues

2006-01-01 Thread Lars Gullik Bjønnes
Martin Vermeer <[EMAIL PROTECTED]> writes: | On Sun, Jan 01, 2006 at 02:17:00PM +0100, [EMAIL PROTECTED] wrote: | > I cannot test it but it looks much | > better. I'd say 'apply it'. | > | > JMarc | | What is this a response to? Your 'let's always repaint the row with the cursor in it' patch I

Re: Various change tracking issues

2006-01-01 Thread Martin Vermeer
On Sun, Jan 01, 2006 at 08:29:23PM +0100, Lars Gullik Bjønnes wrote: > Martin Vermeer <[EMAIL PROTECTED]> writes: > > | On Sun, Jan 01, 2006 at 02:17:00PM +0100, [EMAIL PROTECTED] wrote: > | > I cannot test it but it looks much > | > better. I'd say 'apply it'. > | > > | > JMarc > | > | What is

Re: Various change tracking issues

2006-01-01 Thread Jean-Marc Lasgouttes
| What is this a response to? >Your 'let's always repaint the row >with the cursor in it' patch I >belive. Right. My palm is not very good at quoting messages, sorry. JMarc -- Lgb

Various change tracking issues

2005-12-31 Thread Michael Gerz
Hello, I am sorry to say so but change tracking is not in a good shape at the moment. There are many small problems that, in combination, make change tracking almost unusuable. (If we were able to fix at least a few of them, life would be much easier) I have produced a set of test cases. I

Re: Various change tracking issues

2005-12-31 Thread Lars Gullik Bjønnes
Michael Gerz [EMAIL PROTECTED] writes: | UNDO OF LINE BREAKS FAILS minor can wait. | UNCHANGED PARAGRAPHS CANNOT BE SPLIT a bit more severe. This is basic functionality. | CHANGES CANNOT BE ACCEPTED UNLESS CHANGE TRACKING IS ACTIVATED | 1. New doc | 2. Enter hello world | 3. Place cursor

Re: Various change tracking issues

2005-12-31 Thread John C. McCabe-Dansted
I cannot replicate the following two bugs on lyx-1.4.0pre3-Qt, My system is as follows: Kubuntu 5.10, gcc 4.02, QT 3.3.4, KDE 3.5. What version of LyX and QT (or xforms or...) are you using? Are these MS-Windows only bugs? On Sunday 01 January 2006 01:15, Michael Gerz wrote: SCREEN IS NOT

Re: Various change tracking issues

2005-12-31 Thread Michael Gerz
John C. McCabe-Dansted wrote: On Sunday 01 January 2006 01:15, Michael Gerz wrote: SCREEN IS NOT UPDATED AFTER DELETION 1. New doc 2. Enter hello 3. Activate change tracking 4. Place cursor in front of hello 5. Press delete key = The character is deleted internally but the screen is not

Re: Various change tracking issues

2005-12-31 Thread Michael Gerz
Lars Gullik Bjønnes wrote: | UNCHANGED PARAGRAPHS CANNOT BE SPLIT a bit more severe. This is basic functionality. Yes. | = The line break is visible again with a change bar to the left | (OK!) but there is no way to accept this change because change | tracking is deactivated. IMHO,

Re: Various change tracking issues

2005-12-31 Thread Lars Gullik Bjønnes
Michael Gerz [EMAIL PROTECTED] writes: | John C. McCabe-Dansted wrote: | | On Sunday 01 January 2006 01:15, Michael Gerz wrote: | | SCREEN IS NOT UPDATED AFTER DELETION | | 1. New doc | 2. Enter hello | 3. Activate change tracking | 4. Place cursor in front of hello | 5. Press delete key

Re: Various change tracking issues

2005-12-31 Thread Michael Gerz
Lars Gullik Bjønnes wrote: yes... tracker info is not stored directly in the row. so on a change like this the row signature will stay the same. LFUNS that require an update should perhaps make the row redraw always. Well, I don't know all the details of LyX but to me this does not sound

Re: Various change tracking issues

2005-12-31 Thread Juergen Spitzmueller
Michael Gerz wrote: I am still wondering why the user has to accept all changes before he is allowed to deactivate change tracking.  - If there is a technical reason for it, then the scenrio above will cause trouble sooner or later.  - If not, then LyX restricts the application of change

Re: Various change tracking issues

2005-12-31 Thread Lars Gullik Bjønnes
Juergen Spitzmueller [EMAIL PROTECTED] writes: | Michael Gerz wrote: | I am still wondering why the user has to accept all changes before he is | allowed to deactivate change tracking. |  - If there is a technical reason for it, then the scenrio above will | cause trouble sooner or later. |  

Re: Various change tracking issues

2005-12-31 Thread Lars Gullik Bjønnes
[EMAIL PROTECTED] (Lars Gullik Bjønnes) writes: | Michael Gerz [EMAIL PROTECTED] writes: | | | John C. McCabe-Dansted wrote: | | | | On Sunday 01 January 2006 01:15, Michael Gerz wrote: | | | | SCREEN IS NOT UPDATED AFTER DELETION | | | | 1. New doc | | 2. Enter hello | | 3. Activate

Re: Various change tracking issues

2005-12-31 Thread Michael Gerz
Lars Gullik Bjønnes wrote: Actually I think the simple solution for now is to use Changes::isChange on the row range, and if we have a change, just repaint the row always. This can be accessed through Paragraph::isChanged. So in rowpainter.C: paintPar: If you do this manually, is the

Re: Various change tracking issues

2005-12-31 Thread Michael Gerz
Juergen Spitzmueller wrote: My take on this is as follows: Fix the regressions that have been introduced by the rowSignature patch and leave all other things as they are. If we start to rewrite the change tracker code now (and things like paragraph split/merge will need a major rewrite),

Re: Various change tracking issues

2005-12-31 Thread Martin Vermeer
On Sat, Dec 31, 2005 at 02:47:21PM +0100, Michael Gerz wrote: John C. McCabe-Dansted wrote: On Sunday 01 January 2006 01:15, Michael Gerz wrote: SCREEN IS NOT UPDATED AFTER DELETION 1. New doc 2. Enter hello 3. Activate change tracking 4. Place cursor in front of hello 5. Press

Re: Various change tracking issues

2005-12-31 Thread Martin Vermeer
On Sat, Dec 31, 2005 at 04:34:35PM +0100, Michael Gerz wrote: Lars Gullik Bjønnes wrote: Actually I think the simple solution for now is to use Changes::isChange on the row range, and if we have a change, just repaint the row always. This can be accessed through Paragraph::isChanged.

Re: Various change tracking issues

2005-12-31 Thread Lars Gullik Bjønnes
Martin Vermeer [EMAIL PROTECTED] writes: | Index: rowpainter.C | === | RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/rowpainter.C,v | retrieving revision 1.161 | diff -u -p -r1.161 rowpainter.C | --- rowpainter.C 30 Dec 2005

Re: Various change tracking issues

2005-12-31 Thread Martin Vermeer
On Sat, Dec 31, 2005 at 05:51:47PM +0100, Lars Gullik Bjønnes wrote: Martin Vermeer [EMAIL PROTECTED] writes: | Index: rowpainter.C | === | RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/rowpainter.C,v | retrieving revision

  1   2   >