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

[ios] Improve TTS voice selection #8196

Merged
merged 1 commit into from
May 22, 2024
Merged

[ios] Improve TTS voice selection #8196

merged 1 commit into from
May 22, 2024

Conversation

fabwu
Copy link
Contributor

@fabwu fabwu commented May 16, 2024

This commit allows a user to select more voices (e.g. English (United States), English (India)). Currently, it's only possible to select a subset of the available voices on iOS. For example, if a user selects English as TTS language, an Australian voice is selected because en_AU comes before en_US in the voice list of iOS.

Fixes #6840
Fixes #3222
Fixes #2178
Fixes #7604

Copy link
Member

@biodranik biodranik left a comment

Choose a reason for hiding this comment

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

Cool, thanks!

iphone/Maps/Core/TextToSpeech/MWMTextToSpeech+CPP.h Outdated Show resolved Hide resolved
iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
iphone/Maps/Core/TextToSpeech/MWMTextToSpeech.mm Outdated Show resolved Hide resolved
iphone/Maps/Core/TextToSpeech/MWMTextToSpeech.mm Outdated Show resolved Hide resolved
@fabwu fabwu force-pushed the tts branch 2 times, most recently from c444859 to 9bba62f Compare May 17, 2024 03:05
@fabwu
Copy link
Contributor Author

fabwu commented May 17, 2024

I've changed two more things:

  1. I removed the hard-coded Alex voice for en-US because I wasn't able to select another voice if the Alex voice is installed.

  2. The first entry of the voice list was English (Australian) (see screenshot). I changed it to English (United States). Now the order is English (United States), default voice language depending on the iOS settings and then the selected language.

iphone/Maps/Maps.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
iphone/Maps/Maps.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
iphone/Maps/Maps.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
iphone/Maps/Maps.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
Copy link
Member

@biodranik biodranik left a comment

Choose a reason for hiding this comment

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

Good, thanks!

iphone/Maps/Core/TextToSpeech/MWMTextToSpeech.mm Outdated Show resolved Hide resolved
iphone/Maps/Core/TextToSpeech/MWMTextToSpeech.mm Outdated Show resolved Hide resolved
iphone/Maps/Core/TextToSpeech/MWMTextToSpeech.mm Outdated Show resolved Hide resolved
iphone/Maps/Maps.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
iphone/Maps/Maps.xcodeproj/project.pbxproj Outdated Show resolved Hide resolved
Copy link
Member

@biodranik biodranik left a comment

Choose a reason for hiding this comment

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

A bit of final minor nitpicking )

iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
iphone/Maps/UI/Settings/MWMTTSSettingsViewController.mm Outdated Show resolved Hide resolved
This commit allows a user to select more voices (e.g. English (United States),
English (India)). Currently, it's only possible to select a subset of the
available voices on iOS. For example, if a user selects English as TTS
language, an Australian voice is selected because en_AU comes before en_US in
the voice list of iOS.

Fixes organicmaps#6840
Fixes organicmaps#3222
Fixes organicmaps#2178

Signed-off-by: Fabian Wüthrich <me@fabwu.ch>
Copy link
Member

@biodranik biodranik left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@kirylkaveryn @vng any comments?

@vng vng merged commit d898cf1 into organicmaps:master May 22, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants