Thanks very much for trying this out, Nick.  To be honest, it’s been so long 
since I used a ThinkPad joy nub that I hadn’t considered this at all.  It’s 
good that it seemed just a switch in behavior of key modifiers because it 
really isn’t.  It changes the way panning is done altogether, doing the panning 
in the event handlers instead of firing cmd events.  I think my primary concern 
for non OS X users is that the panning acceleration would seem way out of line 
with expectations, something that's fixable but something I can’t test. 

Also, as I said previously, it changes some longstanding behavior with regard 
to modifier keys and panning direction, but I consider the old behavior wrong 
so I might have to convince folks.  I’m not sure what the timeline is for a 
stable release so I’m not sure whether to wait until after to try to get this 
in.  It really needs some Windows/Linux testing, but there’s not much incentive 
for it.

Garth


> On Jan 10, 2015, at 6:10 AM, Nick Østergaard <[email protected]> wrote:
> 
> I just tried this on linux with a thinkpad nipple, seems to work just
> fine. Without the option enabled I had to use shift to do panning. I
> did not read the patch, but to me it seems like this was just a switch
> in behaivour of using zooming for scroll or having to hit ctrl or
> shift (depinding on what mode was set in settings). It does not seem
> to break any behaviour I usually expect.
> 
> 2014-11-05 9:18 GMT+01:00 Garth Corral <[email protected]>:
>> Thanks for trying this out, guys.  I’d also like to see this get into the 
>> mainline, but I think there are a few hurdles to get over first.  The 
>> pinch-to-zoom should be easy; It’s completely optional and has to be 
>> compiled in so there’s no impact to other platforms, but as I’ve said, it’s 
>> less useful by itself.  The issue for the rest is that it changes some 
>> longstanding behavior, and I’m pretty sure just the idea will annoy some 
>> folks.
>> 
>> I’ve seen very little complaint about the large mousewheel panning steps on 
>> other platforms aside from a couple of ancient bugs.  I think I’d be hard 
>> pressed to convince anyone to change something that isn’t perceived as 
>> broken.  I’ve tried not to do anything egregious here, and I tested this 
>> with a couple of plain old mice, but I have no way to evaluate this on the 
>> other platforms. Does either of you have access to Windows or Linux boxes to 
>> try this on?
>> 
>> To me, the large panning steps make little sense for a mousewheel, and even 
>> less so on a trackpad.  This is a big deal on a platform where every system 
>> in the past several years has sold with input devices with x/y scrolling and 
>> precise deltas (effectively trackpads).  Maybe not so much on other 
>> platforms, I don’t know.
>> 
>> 
>> Thanks again,
>> 
>> Garth
>> 
>> 
>> On Nov 3, 2014, at 6:39 PM, Adam Wolf <[email protected]> wrote:
>> 
>>> I also really like this patch.  Excellent work.
>>> 
>>> Adam Wolf
>>> Cofounder and Engineer
>>> W&L
>>> 
>>> On Sun, Nov 2, 2014 at 10:43 AM, Bernhard Stegmaier 
>>> <[email protected]> wrote:
>>> Hi,
>>> 
>>> works perfect for me… with trackpad and also Magic/Mighty Mouse (without 
>>> pinch-to-zoom for the mice of course).
>>> Tested with the pinch-to-zoom wxWidgets patch on 3.0.2, KiCad changes 
>>> merged into 5249.
>>> 
>>> I would really love to see this getting merged to main line.
>>> It not only makes panning more OSX like, but also is a great usability 
>>> improvement (at least on OSX where nowadays all mice/trackpads do have x/y 
>>> scrolling capability).
>>> 
>>> 
>>> Regards,
>>> Bernhard
>>> 
>>> 
>>> On 31.10.2014, at 18:45, Garth Corral <[email protected]> wrote:
>>> 
>>>> 
>>>> Hi, all
>>>> 
>>>> I’ve pushed a branch containing some changes that I use to make kicad more 
>>>> livable for me on OS X.  This is mostly for Apple Magic Trackpad users, 
>>>> but I’d be curious to hear from mousewheel users on OS X, or even Linux 
>>>> and Windows users if they were inclined to try out these changes.
>>>> 
>>>> The changes are basically in two parts.  The first is optional and 
>>>> requires an additional patch to wxWidgets on OS X, and that kicad be 
>>>> compiled with a feature macro turned on (updated instructions in 
>>>> compiling/mac-osx.txt).  This enables pinch-to-zoom.
>>>> 
>>>> The rest does not require any changes in the way things are built.  This 
>>>> adds a preference for the default mousewheel function and changes the way 
>>>> panning is done; It does not change the way mousewheel zooming behaves 
>>>> (but perhaps should).
>>>> 
>>>> For best results on a trackpad I recommend compiling with the 
>>>> pinch-to-zoom support and checking “Use mousewheel to pan” in the 
>>>> preferences of the various apps.
>>>> 
>>>> If you’re a Mac trackpad user and have a spare few moments, please give 
>>>> these changes a try and let me know how they work for you.  They’re in 
>>>> lp:~gcorral/kicad/osx-trackpad-gestures
>>>> 
>>>> 
>>>> Thanks in advance,
>>>> 
>>>> Garth
>>>> 
>>>> 
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>> Post to     : [email protected]
>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>> More help   : https://help.launchpad.net/ListHelp
>>> 
>>> 
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to     : [email protected]
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help   : https://help.launchpad.net/ListHelp
>>> 
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers
>>> Post to     : [email protected]
>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>> More help   : https://help.launchpad.net/ListHelp
>> 
>> 
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : [email protected]
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
>> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : [email protected]
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp

Attachment: smime.p7s
Description: S/MIME cryptographic signature

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to