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

fix display.get_pointer issues #80

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

barraponto
Copy link
Contributor

i don't really know what's up, but it seems like get_pointer just outputs a tuple

@mrichar1
Copy link
Owner

The latest stable version of GdkDisplay (3.24) definitely has mask - I wonder if this is down to changing API? https://developer.gnome.org/gdk3/stable/GdkDisplay.html#gdk-display-get-pointer

I could accept this if it was set up so that it first tries mask, then falls back to tuple?

@barraponto
Copy link
Contributor Author

I was just trying to fix tests to make my other PR build...
In any case, gdk_display_get_pointer is deprecated, so best course of action would be moving to gdk_device_get_position...

@mrichar1
Copy link
Owner

So I moved to get_pointer since the method I was using of getting the masks went from deprecated to removed, and get_pointer was previously referenced as the new way to get masks.

Now it too is deprecated, and while it suggests get_positionthis provides no access to masks, and I haven't found an alternative route to this through other methods.

It's on my list of things to look at - but unfortunately I don't have much time just now for that...

@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Apr 16, 2022
@stale stale bot closed this Apr 27, 2022
@mrichar1 mrichar1 reopened this Apr 28, 2022
@stale stale bot removed the stale label Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants