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

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

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

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.

@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
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
Projects
None yet
Development

No branches or pull requests

5 participants