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

TTSManager: Fix modality of voice TTS settings dialog #163

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

lumpidu
Copy link
Collaborator

@lumpidu lumpidu commented Feb 21, 2024

The voice TTS settings activity had a wrong modality option when opened. Change intent flags from FLAG_ACTIIVITY_NEW_TASK to FLAG_ACTIVITY_NO_HISTORY.

This will change the behaviour like this:

  • open activity on top of the TTSManager activity
  • close activity when navigating back
  • close activity when resuming by navigating back to Símarómur from another app / activity

This fixes / closes #143

The voiceTTS settings activity had a wrong modality option when opened.
Change intent flags from FLAG_ACTIIVITY_NEW_TASK to FLAG_ACTIVITY_NO_HISTORY.

This will change the behaviour like this:

- open activity on top of the TTSManager activity
- close activity when navigating back
- close activity when resuming by navigating back to Símarómur from another
  app / activity

This fixes #143

Signed-off-by: Daniel Schnell <[email protected]>
@lumpidu lumpidu self-assigned this Feb 21, 2024
@lumpidu lumpidu requested a review from bnika February 21, 2024 19:34
@lumpidu lumpidu added bug Something isn't working bugfix 2.0.x labels Feb 21, 2024
@lumpidu lumpidu merged commit a49961d into master Feb 26, 2024
1 check passed
@lumpidu lumpidu deleted the fix/settings_modality branch February 26, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0.x bug Something isn't working bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Settings - recursion
2 participants