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

Fixing VoiceOver Support #13

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

tbreitenfeldt
Copy link

Implemented two classes: VoiceOver, and SystemVoiceOver. The VoiceOver class leverages Apple Script to directly talk to the currently running instance of VoiceOver, where the SystemVoiceOver class uses NSSpeechSynthesizer to create a new instance of VoiceOver to be used when VoiceOver is inactive.

…n through the speak method. Also changed how VoiceOver.is_active checks the running process by using pgrep.
@TheQuinbox
Copy link

@ctoth, thoughts on merging this? It certainly seems useful for those of us that develop for the Mac.

…d to osascript to avoid script injection. Also changed how Voiceover is determined to be active or not using pgrep.
@tbreitenfeldt
Copy link
Author

Given that I have not found any other working solution, I am ready to merge this pull request if one of the repository owners approves the changes and can merge this. Hopefully some day we are able to find a better solution, however as it is, we will have to live with using apple script and the lag that comes with it until a better solution is found.

@cyrmax
Copy link

cyrmax commented Dec 25, 2022

@ctoth @tspivey , please review and merge this pull request or #16 !
We are trying to create a screenreader for Neovim and we really really need a speech method which would work without VoiceOver on MacOS.

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.

3 participants