Re: visual-line-mode

[Chong Yidong]

David Reitter <address@hidden> writes:

> OK, this works fine now.

Good, I've checked it in (together with a couple other minor fixes).

> The visual-line minor mode is attached again for people to try out
> once you've committed your patch.  Visual-line-mode works fine for me
> with your patches (on 22).
>
> Note that this minor mode works correctly works with variable width
> fonts.  If we had a way to determine the buffer position for given
> pixel coordinates (something like posn-point, without requiring the
> event structure that contains the information), then I could improve
> this package further.

One thing that's not clear to me is whether your minor mode correctly
handles invisible/intangible text and point motion hooks.  One of the
reasons line-move-1 and line-move-finish is so complicated is,
unfortunately, the need to DTRT for these.  I don't see anything
analogous in your code; am I missing something?