Andre Poenitz <[EMAIL PROTECTED]> writes: | On Sun, Nov 14, 2004 at 04:48:14PM +0100, Alfredo Braunstein wrote: >> The attached patch gets rid of the internal xo_ cache of LyXText. > | Things are snapping into place after all. > | I think I owe you something... > >> The last situation where coords are used is when moving the cursor while it >> is off-screen. With the patch, we cleanly abort there ;-) There are at >> least two options: either we do nothing and just recenter the screen, so >> this first keystroke is lost (I think that operations that need coordinates >> are: cursor{up,down},page{up,down},{home,end} plus "selection" brothers) >> *or* we try to do it "right", by first updating the screen and then doing >> the cursor movement. In this case, we would obviously try to optimize for >> the case in which the cursor is on-screen (only the first keystroke is in >> the other case). >> >> I'm ambivalent between the two, mostly because I find myself countlesss >> times pressing cursor-right just to make the screen recenter... Still, I >> don't think that the second option is particularly more work (will see). >> >> Comments? > | The second one sounds cleaner. > | What about being a bit more generous from the beginning and not only | rebreaking paragraphs on screen, but paragraphs touching the range up | to one screen height above and one below. Wouldn't that solve the | problem, too? > | In any case, you should start to consider merging your branch back to | HEAD....
Is it _really_ better already? We should not hurry to get this back into head. -- Lgb