kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #42116
Re: [Patch] Another hotkey processing fix
-
To:
Ian.S.McInerney@xxxxxxxx, Seth Hillbrand <seth@xxxxxxxxxxxxx>
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Wed, 18 Sep 2019 09:25:56 -0400
-
Autocrypt:
addr=stambaughw@xxxxxxxxx; prefer-encrypt=mutual; keydata= mQGiBEM0hxQRBAC2fNh3YOVLu1d5GZ0SbrTNldGiGnCJPLqzEnqFX9v6jmf33TMt6EmSLkl6 Wtfkoj0nVwKxcYmJkA8DX0QAokBkwNIzhSsBzQvthBLIk/5LnPVVKrEXOcL4mUyH1doKlkaE slgJozNa6Av+oavcvD02o1zJOloBbaHlNlyRt7fKswCgtIFlVjWggVH/15KfWk+Qo5JVPbME AIUBAQyL2OAx0n60AWec2WHnO9buHuG0ibtICgUMkE+2MRmYyKwYRdyVwGoIUemFuOyHp0AJ InX4T+vy2E7vkwODqjtMLfIoRkokW74Fi4nrvjlhOAw/vdq/twLbAmR9MOfPTpR4y7kQy1O2 /n+RkkRvh26vTzfbQmrH7cBJhk6aA/9Uwvu3E4zNJgHVZeS0HyWtmR1eOPPRbnkPgJTToX5O KMKzTJI/FX6kT7cFoCamitHrW3BJP4Dx+cMMsa47EGxqVTdbVJ4LjogsXTXxb+0Fn1u4zBdx x3Cer6O7+hqWy7zvpzeC6nSREjqDKa5CgHtv/GLm5uFPOmsjAsnHj2tlBrQmV2F5bmUgU3Rh bWJhdWdoIDxzdGFtYmF1Z2h3QGdtYWlsLmNvbT6IeAQTEQIAOBYhBOffs6CbblRzBkv33BtR cWlZ+CReBQJbFBS2AhsDBQsJCAcCBhUKCQgLAgQWAgMBAh4BAheAAAoJEBtRcWlZ+CReMI8A nRbrLkzp7+c2f0vX7sfg4ICX8LAKAJ9uClo4uJajmZa5zZrL2nKdZlUwIrkCDQRDNIcxEAgA gCru+3/aOC6RCjpvYC72wY+d5SmHphC6yeiV2/mOumyt5MLo/Ps2GznZr11JspqFk5K/Zpvp MMLqqjDZ39+50a2iKRQFJ6NlK+hJWMmj6eJygQrCwYo3Gjc6CqfrqUv+8VSnf/i5sIZmtOVA 4ZjML18MuBvMSsNdVLFJd5HNnYb1iOECpvqdPVh/21LLCEw7MUUGGnHBhCrmk2aJe5hFmcSN g4ldBcXrgMQBwf7aMVoobXBMFDb/IENByXn0llB7Gr2IFMRmNS9/p8s/II1Yl2bTqyX4FSz8 cfn7C9KEz7faZ7wzAcpwHFC/zs3JoAjJ0IEKdNUpIwAlKMzT3CzctwADBQf/cxpG28MKyrqk nNmq/8LQLy+x6FSYXBLjxQz9BiBNYeesDZQ6J5UbL1mjpJzMa5tLZypPYo4bbGyR22hrbyDF K7m6AcVaMIJKl98g4ukMutFfAJyRDaREH5Zl/X1P4u1Z/yaAIy9mKaNbaK1/5djNJ5wCTFen TUgAp9xdc30kGkFDdLJFp5uxDY4P0vaZiZdjUCvDM3Zjv5IzpNOfxVqTUBQNUP/BnnKhkk0p DTD6s3X8S+D0rOtEBQ8K0cwERI/E8EFa8nj0TNw4e2MYGR8wg+SxqJ7z5f0zPY0bO6G9DDFB wYCqzzPWGqdAh9vA5971TAbPERtdFybhkurozp2SfYhJBBgRAgAJBQJDNIcxAhsMAAoJEBtR cWlZ+CResHUAniULLCWiT26ieRTl7N2vS6vBo/DuAJ4m7Ss/gyiW6ybTn1ctDXAUgm2QVQ==
-
Cc:
KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
In-reply-to:
<CACp=VfZNgduaFD8D7Ua9aWMTAEFNVBW+e7O2562CxmrG0tvDrg@mail.gmail.com>
-
Openpgp:
preference=signencrypt
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0
I think we should merge it. I tested it on a new config and my crufty
old config and it seems to work as expected. There may be some issues
for users with multiple actions defined to the same key but that should
be a simple matter of having them reset their key assignment to the
default. If there are no objections, I have this queued up and ready to
merge.
Cheers,
Wayne
On 9/18/19 4:13 AM, Ian McInerney wrote:
> What do we want to do with this patch?
>
> -Ian
>
> On Fri, Sep 6, 2019 at 1:23 AM Seth Hillbrand <seth@xxxxxxxxxxxxx
> <mailto:seth@xxxxxxxxxxxxx>> wrote:
>
> Wow, this bug has been a real annoyance for some time. Thank you for
> tracking down the origin.
>
> I've tested Ian's patch and it works correctly. Allows the 'P' hotkey
> to correctly trigger the Place Power action in eeschema and the Create
> Pin action in LibEdit. Tested with individual applications as well as
> with both applications running at the same time.
>
> Note, however, that when we edit the hotkeys in preferences, no such
> testing of whether a hotkey is handled takes place and we simply show a
> warning that the 'P' hotkey is bound to two separate actions. I'm not
> sure what the course of action here could be other than to separate
> testing of the hotkey overlap between applications.
>
> -Seth
>
> On 2019-09-05 17:21, Ian McInerney wrote:
> > This somehow got lost in my email, but I have now found a case where
> > this
> > is needed in the "wild" (the master branch). For bug
> > https://bugs.launchpad.net/kicad/+bug/1834547, it appears that on
> Linux
> > the
> > hotkey architecture is trying to run the action for place symbol pin
> > instead of the place power action. These two actions are never active
> > on
> > the same editing session, so it is a valid configuration. Applying
> this
> > patch fixes the behavior, and the place power symbol action is
> actually
> > called.
> >
> > -Ian
> >
> > On Mon, Aug 12, 2019 at 9:02 PM Wayne Stambaugh
> <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>
> > wrote:
> >
> >> What is the status of this patch?
> >>
> >> Cheers,
> >>
> >> Wayne
> >>
> >> On 8/8/19 7:16 PM, Ian McInerney wrote:
> >> > In the current framework, if more than one global actions share
> the same
> >> > hotkey (even if they are not all active in the current tool
> manager),
> >> > the dispatcher will only choose the final action (in what seems
> to be
> >> > alphabetical order) to run. I think that the correct behavior
> should
> >> > instead be to loop through all global actions that have the
> hotkey until
> >> > one handles it.
> >> >
> >> > The attached patch implements this change.
> >> >
> >> > -Ian
> >> >
> >> > _______________________________________________
> >> > Mailing list: https://launchpad.net/~kicad-developers
> >> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto: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
> <mailto: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
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help : https://help.launchpad.net/ListHelp
>
Follow ups
References