From: Christophe CURIS <[email protected]> The original choice may have looked mathematically correct, but it was actually counter-intuitive and opposite to what every other application do with sliders.
Signed-off-by: Christophe CURIS <[email protected]> --- WINGs/wslider.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/WINGs/wslider.c b/WINGs/wslider.c index a18987a..cacf57c 100644 --- a/WINGs/wslider.c +++ b/WINGs/wslider.c @@ -410,16 +410,16 @@ static void handleActionEvents(XEvent * event, void *data) switch (event->type) { case ButtonPress: - if (event->xbutton.button == WINGsConfiguration.mouseWheelUp && !sPtr->flags.dragging) { - /* Wheel up */ + if (event->xbutton.button == WINGsConfiguration.mouseWheelDown && !sPtr->flags.dragging) { + /* Wheel down */ if (sPtr->value + 1 <= sPtr->maxValue) { WMSetSliderValue(sPtr, sPtr->value + 1); if (sPtr->flags.continuous && sPtr->action) { (*sPtr->action) (sPtr, sPtr->clientData); } } - } else if (event->xbutton.button == WINGsConfiguration.mouseWheelDown && !sPtr->flags.dragging) { - /* Wheel down */ + } else if (event->xbutton.button == WINGsConfiguration.mouseWheelUp && !sPtr->flags.dragging) { + /* Wheel up */ if (sPtr->value - 1 >= sPtr->minValue) { WMSetSliderValue(sPtr, sPtr->value - 1); if (sPtr->flags.continuous && sPtr->action) { -- 2.1.1 -- To unsubscribe, send mail to [email protected].
