kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #21054
Re: [PATCH] Support of horizontal scrolling events in pcbnew and 3d-viewer
On 11/2/2015 3:38 PM, Garth Corral wrote:
>
>> On Nov 2, 2015, at 12:20 PM, Wayne Stambaugh <stambaughw@xxxxxxxxx
>> <mailto:stambaughw@xxxxxxxxx>> wrote:
>>
>>
>>
>> On 11/2/2015 3:00 PM, Garth Corral wrote:
>>>
>>>> On Nov 2, 2015, at 10:25 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx
>>>> <mailto:stambaughw@xxxxxxxxx>> wrote:
>>>>
>>>> On 11/1/2015 9:42 AM, Bernhard Stegmaier wrote:
>>>>> If this capability should be added now, it should be done
>>>>> consistent for all platforms.
>>>>> We had this discussion for OS X already way back, but back then
>>>>> nobody seemed to be interested in adding also for Windows/Linux…
>>>>>
>>>>> Garth made some changes where you even could switch between mouse
>>>>> wheel X/Y panning and normal behaviour via preferences menu.
>>>>
>>>> If I remember correctly, Garth's changes broke the existing behavior
>>>> which made it a non-starter. If this fix didn't break the existing
>>>> behavior then I wouldn't have a problem accepting it but not until after
>>>> the stable release. It's getting too late for changes that could
>>>> potentially introduce new bugs. Our mouse event handling code is a mess
>>>> and very fragile so adding more changes to it is just inviting problems.
>>>>
>>> To be clear, it did not break the behavior. It was a conscious
>>> decision to change the currently broken behavior. I knew it would
>>> not be received well, but whatever.
>>
>> Refresh my memory. What is broken about the current behavior? I
>> thought this was adding new behavior to handle track pad devices which
>> should not have impacted the mouse scroll wheel panning. Are you saying
>> that this could not be implemented without changing the current panning
>> behavior?
>>
> I changed the behavior with regard to the panning direction and modifier
> keys to be more in line with UI convention and also to fix the incorrect
> left/right panning with the mousewheel. When you’re rolling a
> mousewheel up and down you might not notice or care which left/right
> direction that translates to when modified, but when you’re actually
> scrolling in four directions on a trackpad it becomes obvious. So it’s
> unlikely that most mouse users would notice that there is any issue, but
> trackpad users will.
>
> The point is sort of moot now anyway. I’m not currently in a position
> to commit that code. I think that Bernhard had some parallel changes in
> his tree that accomplish much of the same things. Perhaps that can go
> in at some point. I still occasionally see posts on forums & lists
> complaining of OS X usability issues around trackpad use, it would be
> good to sort that out. I’m trying to work with my employer so that I’ll
> be able to contribute again in the future but I’m pretty much out at
> this point.
>
> Garth
>
Thanks for the update. Maybe Bernhard can get this in shape for soon
after the stable release. I hope your employer gives you some leeway to
contribute to KiCad. It would be nice to have another OSX dev to help out.
Wayne
References