kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #27316
Re: [PATCH] mousewheelpan + ctrl = zooming
Hi Brano and Orson!
Code of the zooming was not modified in this patch. But I noticed, that
there is present MAC-specific block:
common/view/wx_view_controls.cpp (+147):
>
> #ifdef __WXMAC__
> // The following is to support Apple pointer devices (MagicMouse &
> // Macbook touchpad), which send events more frequently, but with
> smaller
> // wheel rotation.
> //
> // It should not break other platforms, but I prefer to be safe
> than
> // sorry. If you find a device that behaves in the same way on
> another
> // platform, feel free to remove #ifdef directives.
> if( timeDiff > 0 && timeDiff < 100 && std::abs( rotation ) < 20 )
> {
> aEvent.Skip();
> return;
> }
> #endif
>
Maybe is problem in it?
--
Regards, Konstantin.
2017-01-23 11:38 GMT+02:00 Brano Panak <bpanak@xxxxxxxxx>:
> Hi Orson,
>
> this occurs when touchpad panning enabled. When TP panning disabled wheel
> works ok (withou need to push ctrl).
>
> When TP panning enabled it zoom in work ok. When zooming out it look to me
> that sw zooms out and immediately zooms in again.
>
> I am using wireless apple mouse which has both (vertical and horizontal)
> "wheels".
>
> When trying to zoom in/out with ctrl+ horizontal "wheel" all works ok.
>
> If i remember correctly this was working ok, befor latest "wheel" patch.
>
> regards
>
> Brano
>
> On 20/01/17 08:52, Maciej Sumiński wrote:
>
> Hi Brano,
>
> Are the problems occurring when touchpad panning is enabled? Is
> everything fine once you disable it?
>
> Would you give more details about zoom out being problematic? Does it
> zoom out too quickly, is it choppy?
>
> Regards,
> Orson
>
> On 01/19/2017 10:52 PM, Brano Panak wrote:
>
> Today i checked behavior on osx with version: (2017-01-18 revision
> 2eb840b)-master
> and behavior in pcbnew during zooming with wheel+ctrl(cmd on mac) on
> apple mouse is very erratic.
>
> Zoom in works ok, zoom out is very problematic.
>
> Eeschema looks ok, pcbnew with zooming and scrolling on touchpad with 2
> fingers looks good.
>
> regards
>
> Brano
>
>
> On 17/01/17 18:17, Константин Барановский wrote:
>
> In attachment placed new patch, that contains both previous patches
> and in mousewheelpan mode, with pressed Shift key, does horizontal
> scrolling.
>
> So, in common, we will get:
>
> * mousewheelpan disabled:
> - mousewheel = zooming;
> - mousewheel + ctrl = horizontal scrolling;
> - mousewheel + shift = vertical scrolling;
>
> * mousewheelpan enabled:
> - touchpad two finger scrolling = pan;
> - mousewheel = vertical scrolling;
> -> mousewheel (touchpad two finger scrolling) + ctrl = zooming;
> -> mousewheel (touchpad two finger scrolling) + shift = horizontal
> scrolling.
>
> This patch adds two last options and decreases the pan step in
> 3d-viewer to be more comfortable.
> It works in eeschema, pcbnew (legacy, openGL, cairo), 3d-viewer,
> gerbview.
>
> 2017-01-17 11:27 GMT+02:00 Maciej Sumiński <maciej.suminski@xxxxxxx<mailto:maciej.suminski@xxxxxxx> <maciej.suminski@xxxxxxx>>:
>
> Hi Konstantin,
>
> Your patches were not ignored, the problem was neither me nor Wayne
> could fully apply the patches in their original version. Thank you
> for
> correcting this.
>
> I still hold my previous remark about wheel scroll and shift+wheel
> scroll doing the same thing when touchpad panning is enabled.
> Instead,
> shift+wheel scroll could perform horizontal scrolling in touchpad
> panning mode.
>
> Regards,
> Orson
>
> On 01/16/2017 07:38 PM, Константин Барановский wrote:
> > Thank you, Wayne, for your response. I'm attached the checked
> patches.
> >
> > 2017-01-16 20:07 GMT+02:00 Wayne Stambaugh <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx> <stambaughw@xxxxxxxxx>>:
> >
> >> You are not being ignored. I can't speak for all of the lead
> developers
> >> but I've been really busy so pretty much everyone has been getting
> >> ignored by me. It's not intentional, it's just the reality of my
> >> current work load.
> >>
> >> Patch is giving me an unexpected eof with
> 3d_viewer-pan_step.patch.
> >>
> >> Did you address Bernhard's concern about the behavior of the GAL
> >> canvases as well as the legacy canvas? I don't remember seeing
> anything
> >> but I may have missed it.
> >>
> >> On 1/16/2017 11:29 AM, Константин Барановский wrote:
> >>> Hello. I'm sorry for disturbing you, but I'm not understand
> why my
> >>> messages are ignored. Proposed feature not needed for no one,
> except me?
> >>> Or I'm made something wrong? Please, give any comment.
> >>>
> >>> 2017-01-06 12:03 GMT+02:00 Константин Барановский
> >>> <baranovskiykonstantin@xxxxxxxxx
> <mailto:baranovskiykonstantin@xxxxxxxxx> <baranovskiykonstantin@xxxxxxxxx>
> <mailto:baranovskiykonstantin@xxxxxxxxx <baranovskiykonstantin@xxxxxxxxx>
> <mailto:baranovskiykonstantin@xxxxxxxxx> <baranovskiykonstantin@xxxxxxxxx>
> >>>> :
> >>>
> >>> 2016-11-23 0:22 GMT+02:00 Maciej Sumiński
> <maciej.suminski@xxxxxxx <mailto:maciej.suminski@xxxxxxx> <maciej.suminski@xxxxxxx>
> >>> <mailto:maciej.suminski@xxxxxxx <maciej.suminski@xxxxxxx>
> <mailto:maciej.suminski@xxxxxxx> <maciej.suminski@xxxxxxx>>>:
> >>>
> >>> I could not apply the second patch, it gives me
> "unexpected end
> >>> of file
> >>> in patch" error. Would you verify the file?
> >>>
> >>>
> >>> I downloaded both patches and checked them, they looks
> good for me.
> >>> I do not got any problems or errors.
> >>>
> >>>
> >>
> >
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/%7Ekicad-developers> <https://launchpad.net/%7Ekicad-developers>
> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx> <kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/%7Ekicad-developers> <https://launchpad.net/%7Ekicad-developers>
> > More help : https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp> <https://help.launchpad.net/ListHelp>
> >
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/%7Ekicad-developers> <https://launchpad.net/%7Ekicad-developers>
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx> <kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/%7Ekicad-developers> <https://launchpad.net/%7Ekicad-developers>
> More help : https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp> <https://help.launchpad.net/ListHelp>
>
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
>
References
-
[PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2016-11-10
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2016-11-11
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Wayne Stambaugh, 2016-11-11
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Bernhard Stegmaier, 2016-11-11
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Wayne Stambaugh, 2016-11-11
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2016-11-12
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2016-11-12
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Maciej Sumiński, 2016-11-22
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2017-01-06
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2017-01-16
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Wayne Stambaugh, 2017-01-16
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2017-01-16
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Maciej Sumiński, 2017-01-17
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Константин Барановский, 2017-01-17
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Brano Panak, 2017-01-19
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Maciej Sumiński, 2017-01-20
-
Re: [PATCH] mousewheelpan + ctrl = zooming
From: Brano Panak, 2017-01-23