#81 File dialog wheel support

development
closed-accepted
Eddy De Greef
Program (79)
5
2003-05-02
2003-04-05
Duane Clark
No

Adds wheel mouse support to the file dialog.

Discussion

  • Thorsten Haude
    Thorsten Haude
    2003-04-05

    Logged In: YES
    user_id=119143

    Sounds interesting.

    Sourceforge is broken and does not accept files on initial
    submit. Would you attach the patch please?

     
  • Duane Clark
    Duane Clark
    2003-04-05

    File dialog wheel support

     
    Attachments
  • Duane Clark
    Duane Clark
    2003-04-05

    Logged In: YES
    user_id=750500

    Okay, here is another attempt to attach it.

     
  • Thorsten Haude
    Thorsten Haude
    2003-04-05

    Logged In: YES
    user_id=119143

    Works good.

    Is it possible to change the number of lines scrolled, maybe
    with a resource?

     
  • Duane Clark
    Duane Clark
    2003-04-05

    Logged In: YES
    user_id=750500

    I tried several methods in an attempt to scroll by more than
    one. The only one I was able to make work was to call the
    XtCallActionProc multiple times in a loop in the
    scrollUpAP() and scrollDownAP(). That actually seems to work
    fine, so it probably would be a good idea to make that a
    resource. I'll give it a shot.

     
  • Duane Clark
    Duane Clark
    2003-04-06

     
    Attachments
  • Duane Clark
    Duane Clark
    2003-04-06

    Logged In: YES
    user_id=750500

    After trying to figure out resources in nedit, I finally
    decided to just do it the same way as was done for the text
    widget, except I hardcoded 3 instead of 5 as the normal
    scroll line length.

     
  • Thorsten Haude
    Thorsten Haude
    2003-04-07

    Logged In: YES
    user_id=119143

    Haven't tried the new version, but the code looks fine to me.

    Yet another thing: This should also be applied to the dialog
    created by the macro function list_dialog(); do you see an
    easy way to do it, maybe some function
    addWheelTranslationToXmList()? (If not, I will have a look
    at it. Shouldn't be too hard now that I can copy the pattern.)

    Uh. In fact, this should be applied to most if not all
    XmLists throughout the application. Should we derive our own
    List class? (Why do these things have the tendendancy to
    grow like cancer the more you think about it?)

    In any case, your patch is useful as it is, this has been
    requested several times in the past and should be put in CVS
    asap. Thanks!

     
  • Duane Clark
    Duane Clark
    2003-04-07

    Logged In: YES
    user_id=750500

    I've never actually written macros, and so don't really know
    how to use those functions. So I think I'll let you (or
    whoever) go ahead and add that part :-)

     
  • Thorsten Haude
    Thorsten Haude
    2003-04-07

    Logged In: YES
    user_id=119143

    No, no, I meant the C function which sits behind the macro
    function. If you are interested, have a look at
    listDialogMS() in macro.c (which seems to be the right place
    to do it) or look at my (hopefully) forthcoming patch.

     
  • Eddy De Greef
    Eddy De Greef
    2003-04-08

    Logged In: YES
    user_id=73597

    Better mouse wheel support is definitely something we must have.
    Therefore I've taken the liberty to generalize your patch
    such that it works for all scrolled widgets (not only lists)
    in NEdit (except for the text area, which already had support).
    I haven't tested it very well, but it's a quite simple
    add-on and I don't expect many problems. Any comments?

     
  • Eddy De Greef
    Eddy De Greef
    2003-04-08

    Generalized mouse wheel support

     
    Attachments
  • Duane Clark
    Duane Clark
    2003-04-08

    Logged In: YES
    user_id=750500

    Mainly that I never realized there were so many list widgets
    in nedit ;) The patch works great for me.

     
  • Eddy De Greef
    Eddy De Greef
    2003-04-11

    Added mouse wheel support in selection dialogs

     
    Attachments
  • Eddy De Greef
    Eddy De Greef
    2003-05-02

    Logged In: YES
    user_id=73597

    The patch is committed to CVS.

     
  • Eddy De Greef
    Eddy De Greef
    2003-05-02

    • assigned_to: nobody --> edg
    • status: open --> closed-accepted