Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow unbinding of Gestures from super classes by None value in __gestures dict #4240

Closed
nvaccessAuto opened this issue Jul 1, 2014 · 4 comments
Closed

Allow unbinding of Gestures from super classes by None value in __gestures dict #4240

nvaccessAuto opened this issue Jul 1, 2014 · 4 comments
Labels
bug
Milestone
2014.3

Comments

@nvaccessAuto
Copy link

Reported by mdcurran on 2014-07-01 03:21
Sometimes a gesture may be bound to a script on a supr class, yet a sub class might not want it to be bound. The sub class could bind its own script that just did a gesture.send(), but allowing the None keyword as the script name is less work for the author. An example where this is needed is to unbind the tab key in the spellCheck field for MS Word, but that NVDAObject inherits from the WordDocument NVDAObject which binds the tab key for announcing table navigation / indenting / form field navigatin.
@nvaccessAuto
Copy link
Author

Comment 1 by mdcurran on 2014-07-01 03:22
An issue was also identified for one implementation of __gestures where the mro of the classes was being walked from sub to super, it should be from super to sub, so that sub class bindings always override super class bindings. This should also be fixed.

@nvaccessAuto
Copy link
Author

Comment 2 by Michael Curran <mick@... on 2014-07-02 01:25
In [1176ebf]:

Merge branch 't4240' into next. Incubates #4240

Changes:
Added labels: incubating

@nvaccessAuto
Copy link
Author

Comment 3 by Michael Curran <mick@... on 2014-07-19 22:47
In [3af5bec]:

Merge branch 't4240'. Fixes #4240

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto
Copy link
Author

Comment 4 by mdcurran on 2014-07-19 22:50
Changes:
Milestone changed from next to 2014.3

@nvaccessAuto nvaccessAuto added this to the 2014.3 milestone Nov 10, 2015
@nvaccessAuto nvaccessAuto mentioned this issue Nov 10, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
2014.3
Development

No branches or pull requests
1 participant
@nvaccessAuto