8000 cursor doesn't change on keypress (GTKAgg backend) · Issue #3447 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]
Skip to content

cursor doesn't change on keypress (GTKAgg backend) #3447

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.

Sign up for GitHub

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
Closed
kaufman opened this issue Sep 1, 2014 · 6 comments
Closed

cursor doesn't change on keypress (GTKAgg backend) #3447

kaufman opened this issue Sep 1, 2014 · 6 comments
Labels
status: needs confirmation
Milestone
v1.4.x

Comments

@kaufman
Copy link
Contributor
kaufman commented Sep 1, 2014

When changing the tool mode via keyboard shortcut (for example 'o' to Zoom), the cursor doesn't change immediately. Instead, the mouse has to be moved for the cursor to change. This behavior occurs on the GTKAgg backend, not sure about any others.
@jenshnielsen jenshnielsen added this to the v1.4.x milestone Sep 1, 2014
@kaufman
Copy link
Contributor Author
kaufman commented Sep 1, 2014

The backends MacOSX and Qt4Agg also have this problem.

@tacaswell
Copy link
Member

This is probably related to the draw vs draw_idle issues. For the most part we try to let the gui-framework decide when to re-draw anything on the screen.

I suspect that we are not invalidating the right things when we change the cursor.

@fariza
Copy link
Member
fariza commented Sep 5, 2014

The problem is that in the toolbar the set_cursor method is called wihtin the mouse_move method.
This happens in all the backends that I tested (QT, GTK3, TK) but from the code I guess it happens everywhere.

I'll see what I can do to fix this.

@fariza
Copy link
Member
fariza commented Sep 5, 2014

@tacaswell How do I know where the cursor is, if the event is a keystroke?

Edit.
Sorry, stupid question it's included in the event

@fariza fariza mentioned this issue Sep 5, 2014
Merged
@fariza
Copy link
Member
fariza commented Sep 5, 2014

Done PR #3474

@fariza
Copy link
Member
fariza commented Sep 9, 2014

@tacaswell please close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status: needs confirmation
Projects
None yet
Milestone
v1.4.x
Development

No branches or pull requests
5 participants
@tacaswell @jenshnielsen @kaufman @anntzer @fariza
0