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

MacOS backend modifier keys #1259

Closed
pelson opened this issue Sep 15, 2012 · 8 comments
Closed

MacOS backend modifier keys #1259

pelson opened this issue Sep 15, 2012 · 8 comments

Comments

@pelson
Copy link
Member

pelson commented Sep 15, 2012

The MacOS backend does not currently have modifier key support such as that available in TkAgg, GtkAgg, WxAgg, QT4Agg etc.

A simple test: Create a figure and try hitting ctrl+w

Note: Follows on from #851.

@dmcdougall
Copy link
Member

I've been looking into this. The problem is the FigureCanvasMac class does not implement functions such as key_press_event, like the GTK backend does. My initial assessment is that this, although would be nice to have, will take a little time. It probably shouldn't stop the release candidate, but it should make it into a bugfix release, perhaps.

@pelson
Copy link
Member Author

pelson commented Sep 17, 2012

It probably shouldn't stop the release candidate, but it should make it into a bugfix release, perhaps.

There is no urgency on this IMHO. It is a thing that we would love to have done, but ultimately, will only get done if somebody really needs it (and is motivated to do it!). Personally, I wouldn't be concerned if this didn't get in until 1.3.

@mdehoon
Copy link
Contributor

mdehoon commented Dec 12, 2012

key_press_event for FigureCanvasMac is implemented in in the C code in _macosx.m in the keyDown method. Probably it's not difficult to add modifiers to it. Any volunteers?

@dmcdougall
Copy link
Member

I started a local branch for this and never got round to finishing it. I'm happy to volunteer to do it, but I'd like to get some of my current pull requests completed before creating more unfinished pieces of work.

@dmcdougall
Copy link
Member

(Please don't let that prevent anybody else from writing a patch. I can't find my branch anyway!)

@cimarronm
Copy link
Contributor

Added a patch for this in #1841

dmcdougall added a commit that referenced this issue Mar 27, 2013
Fixes issue #1259 - Added modifier key handling for macosx backend
@dmcdougall
Copy link
Member

#1841 merged; closing.

@pelson
Copy link
Member Author

pelson commented Mar 27, 2013

Wooho thanks @cimarronm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants