Re: lynx-dev [PATCH 2.8.3.dev4] Scrollbar support

[Ilya Zakharevich]

>> This patch makes it possible to compile lynx with -DUSE_SCROLLBAR
>> option.  Given this option, command-line options -scrollbar and
>> -scrollbar_arrow as well as corresponding LYNX_CFG options are enabled.

> Nice.
>
> I don't remember the details of mouse handling in lynx - does it allow to
>distinguish buttons?

Absolutely.

> If yes, I'd like the following to be added:
>
>If middle button was clicked on the scrollbar, the center of bar should move
>to the click position (and the document to be scrolled correspondingly) - a la
>Motif and Tk widgets -  this gives ability of direct positioning.

I was planning to add it either to Middle-button, or to double-click
of button-1 (may be better, since middle button usually posts menu).
The reason to not put it into the first implementation is that there
is no "position at line 735" lynx-keyaction.  After adding this
keyaction (with 735 kept in a global variable) such a change is
trivial.

Now Klaus's questions:

 a) I do not know which mouse ports would work with the scrollbar
    patch.  Instead of defining one low-level function with
    system-specific implementation ("get xy-position of the last
    click"), the high-level mouse-handling code in wrapped in
    #defines, and duplicated in its entirety separatedly for each
    mouse-handling system.  This makes predicting the behaviour on
    different ports very hard.

    I tested the patch with ncurses only.  It would not *break* other
    ports, but some additional work may be needed.

 b) The only function of scrollbar without mouse support is as visual
    position indicator (in addition to "(p7 of 15)", which has a very
    different granularity).

Hope this helps,
Ilya