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

Add support for mouse pointer interaction introduced in iOS 13.4 #35

Merged
merged 2 commits into from Mar 31, 2020

Conversation

narbs
Copy link
Contributor

@narbs narbs commented Mar 29, 2020

When a mouse is connected to an iPad, the emulated mouse pointer will follow the iOS mouse pointer movements. Change is backward compatible to earlier iOS versions < 13.4 without mouse pointer support.

It makes the emulator much nicer to use with a mouse on an iPad.

Unfortunately I tried but couldn't yet find a way to hide the iOS mouse cursor so the two cursors are on-screen at the moment.

This has been tested working on iOS 13.4 and 13.3.1.

…en a mouse is connected to an iPad, the emulated mouse pointer will follow the iOS mouse pointer movements. Change is backward compatible to earlier iOS versions < 13.4 without mouse pointer support
Copy link
Owner

@zydeco zydeco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I've added a suggestion to hide the cursor.

Mini vMac/ViewController.m Outdated Show resolved Hide resolved
… running on iOS 13.4


Uses the UIPointerStyle hiddenPointerStyle to set hide the iOS cursor in the pointerInteraction:styleForRegion delegate method. Also returns the defaultRegion in the pointerInteraction:regionForRequest:defaultRegion delegate method instead of returning nil

Co-Authored-By: Jesús A. Álvarez <zydeco@namedfork.net>
@narbs
Copy link
Contributor Author

narbs commented Mar 31, 2020

You're welcome. I tried this approach but forgot to return defaultRegion instead of nil - that's why it didn't work for me.

Thank you! It looks so good now...

@narbs narbs closed this Mar 31, 2020
@narbs narbs reopened this Mar 31, 2020
@narbs
Copy link
Contributor Author

narbs commented Mar 31, 2020

Sorry didn't mean to close it... back to you

@WeedyWeedSmoker
Copy link

WeedyWeedSmoker commented Jun 1, 2020

@narbs @zydeco That commit is Great!

Could this be achieved on BasiliskII for iOS?

Thanks to both of you…


BTW, I have a more up to date fork of @zydeco's great BasiliskII port, including a fix from @rakslice for recent iOS: https://github.com/WeedyWeedSmoker/macemu

@zydeco
Copy link
Owner

zydeco commented Jun 3, 2020

@narbs @zydeco That commit is Great!

Could this be achieved on BasiliskII for iOS?

Thanks to both of you…

BTW, I have a more up to date fork of @zydeco's great BasiliskII port, including a fix from @rakslice for recent iOS: https://github.com/WeedyWeedSmoker/macemu

I added it here: zydeco/macemu@49c5c7b

@WeedyWeedSmoker
Copy link

@zydeco Silly me I merged my fork into master, and never thought to check the ios branch…

Thank you

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

Successfully merging this pull request may close these issues.

None yet

3 participants