← Back to team overview

kicad-developers team mailing list archive

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