mirror of
https://github.com/SillyTavern/Extension-Speech-Recognition.git
synced 2026-10-15 21:42:08 +00:00
[PR #8] [MERGED] Turn off the mic when switching providers #17
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
SillyTavern/Extension-Speech-Recognition#17
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/SillyTavern/Extension-Speech-Recognition/pull/8
Author: @Risenafis
Created: 7/12/2024
Status: ✅ Merged
Merged: 7/13/2024
Merged by: @Cohee1207
Base:
main← Head:stop-mic📝 Commits (2)
4983c14Turn off the mic when switching providers5aa7f28Stop the recording when switching to "None"📊 Changes
2 files changed (+59 additions, -13 deletions)
View changed files
📝
browser.js(+10 -6)📝
index.js(+49 -7)📄 Description
When switching the provider while the microphone is in recording mode, the microphone display and recording status became inverted. I have fixed this by stopping the recording of the previous provider when loading a new provider.
I understand that the current microphone icon implementation indicates possible actions, but personally, I feel it is more natural to display the current recording status (similar to the display method in many messaging apps like Zoom and Teams).
That said, ChatGPT (Android app) is close to the current SillyTavern implementation, so I think there may be some demand for it (although it differs in that the big stop button is displayed separately and prominently at the bottom).
Still, I made the title toggle "Click to talk" / "Click to stop talking" so that it might be a bit clearer.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.