Andre Poenitz wrote:

> Things are snapping into place after all.

I'm as surprised as you ;-) 

> I think I owe you something...

Nah... just putting toghether some pieces here and there... you did all the
hard work.

> The second one sounds cleaner.

Cool, I've done that.

> 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?

Well, in the patch we draw just one off-screen paragraph above and one below
(with nullpainter btw), this is enough for cursorUp,Down and also for
pgup/down in their new form (pgup is setCursorFromCoords(x,0) + cursorUp()
+ fitcursor). But this doesn't solve the problem of the cursor being
initially off-screen (*without* cursor-follows-scrollbar of course), nor it
would be solved by painting another screenful up and down. For that we need
an initial fitCursor. I've just commited a patch with this.

> In any case, you should start to consider merging your branch back to
> HEAD....

I also think so. OTOH, I'm ok also keeping it here for a while if people are
reluctant and want to test it further... but IMO it's already far more
useable than HEAD.

Alfredo


Reply via email to