Skip to content

Merge transcription sound toggle into picker with None option#168

Merged
altic-dev merged 2 commits intoaltic-dev:mainfrom
moogman:b/merge-sound-toggle
Feb 23, 2026
Merged

Merge transcription sound toggle into picker with None option#168
altic-dev merged 2 commits intoaltic-dev:mainfrom
moogman:b/merge-sound-toggle

Conversation

@moogman
Copy link
Contributor

@moogman moogman commented Feb 19, 2026

Description

Simplifies the Transcription Sounds settings by removing the separate on/off toggle and using a single sound picker that includes None to disable playback.

Type of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update

Related Issues

Closes #(issue number)

Testing

  • Tested on Intel/Apple Silicon Mac
  • Tested on Apple Silicon Mac
  • Tested on macOS [Sonoma]
  • Ran linter locally: brew install swiftlint && swiftlint --strict --config .swiftlint.yml
  • Ran formatter locally: brew install swiftformat && swiftformat --config .swiftformat Sources

Screenshots / Video

Add screenshots or Video recording of the app after you have made your changes
Screenshot 2026-02-19 at 11 49 21

@moogman moogman marked this pull request as ready for review February 19, 2026 11:51
@moogman moogman force-pushed the b/merge-sound-toggle branch from 52edfc9 to d23aa9f Compare February 19, 2026 12:02
@altic-dev
Copy link
Owner

Good one! if you can rebase to main as I've merged my earlier PR for 1.5.8, we can bring this onto 1.5.9 :) Thanks for catching this!

@moogman
Copy link
Contributor Author

moogman commented Feb 21, 2026

Done :)

@altic-dev altic-dev merged commit 633a0f6 into altic-dev:main Feb 23, 2026
1 check 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

Development

Successfully merging this pull request may close these issues.

2 participants