-
Notifications
You must be signed in to change notification settings - Fork 21
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
2918: Add tts on web #2985
base: main
Are you sure you want to change the base?
2918: Add tts on web #2985
Conversation
…d unnecessary mock
…le from ttsContext
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work 🚀
Tested this on chrome and firefox.
Firefox voice is really crappy, but chrome is really nice 🚀 Maybe we add hint, that some browsers better support this feature? Maybe @hauf-toni can share some input, how to handle this best from a UI perspective.
Found a few bug:
issue 1:
- Open this page: http://localhost:9000/augsburg/de/willkommen/willkommen-in-augsburg-2
- Click "read aloud"
- Click "play"
- Click on "Weiter"
Result: The reader starts reading out the beginning of the current sentance, when clicking "weiter" for the first time. When clicking "weiter" a second time, it correctly jumps to the next sentance
Expected result: When clicking "weiter" for the first time, it should directly jump to the next sentance.
issue 2:
When opening this page: http://localhost:9000/augsburg/de/willkommen
the "read aloud" button is displayed, but does nothing when clicked.
Expected: Should either not be displayed or does something when clicked
issue 3:
Open this page: http://localhost:9000/augsburg/de/behoerden-beratung/behoerden/auslaenderbehoerde
Click "next" to jump to the last sentance
in this line "Weitere Informationen finden Sie unter [hier]" the reader switches to the "start"-mode, but there is still this whole contact blog that is read
issue 4:
When clicking pause I see the initial view of the tts-player, with the first word of the page in the heading. I think this is confusing, as clicking play does not continue reading from the beginning. I would expect that the player just changes the pause button to showing a play button.
what i get when i click pause
what i expected
issue 5:
on firefox when the tts reaches the end of the text, the player does not reset to start as it is for other browsers.
Can you please mention which browser you used for each issue? |
I mostly used Chrome, sometimes I double checked with Firefox |
@bahaaTuffaha i just found one issue.
![]() And we may create a issue for tts voices improvement for safari (or disable it for safari) its really horrible |
native/src/hooks/useTtsPlayer.ts
Outdated
@@ -17,8 +16,7 @@ const useTtsPlayer = (model?: PageModel | LocalNewsModel | TunewsModel | undefin | |||
useFocusEffect( | |||
useCallback(() => { | |||
if (model && model.content.length > 0) { | |||
const content = parseHTML(model.content) | |||
const sentences: string[] = segment(languageCode, content) | |||
const sentences: string[] = segmentation(languageCode, addingPeriodsToDom(model.content)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❓ Why do we need periods here now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm .. not actually needed for native but keeping the web and native consistent.
In general I think tts for native works differently and as I remember it didn't need a period to figure out when to stop unlike web.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could reproduce on firefox linux.
Actually, I can't. Everything works as expected for me and the tts correctly jumps to the next sentence after each utterance/sentence, even without adding extra periods. Could you please test this and add concrete info on how to reproduce this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I can't. Everything works as expected for me and the tts correctly jumps to the next sentence after each utterance/sentence, even without adding extra periods. Could you please test this and add concrete info on how to reproduce this?
Comment out the following lines at ttsDomManipulation.ts
:
appendPeriod(paragraphs)
appendPeriod(listItems)
At firefox (anything doesn't have a full stop at the end will continue reading like lists):
http://localhost:9000/muenchen/en/everyday-life/information-on-insurance-contracts-shopping-invoices-reminders-finances/bank-account
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please check #3150 if this works as expected for you there? With my proposal I can't find any issues.
2918: Refactoring and fixes for tts web
Short description
Adding text-to-speech functionality for web.
Proposed changes
TtsPlayer.tsx
will receive content and title fromuseTtsPlayer.ts
hook then it will process content as following:<p>
or<li>
tag if there is non at the end.sentencex
to support most languages.TtsPlayer.tsx
using EasySpeech library for web speech api .. why I used it instead of directly using the native api read here: https://github.com/leaonline/easy-speech/blob/master/FAQ.md#why-does-this-library-exists-if-i-can-use-tts-natively-in-the-browserWeb Speech API has some limitation on devices like Android I can't pause so I implemented a way to increment
currentSentencesIndex
at each sentence so pausing will be depend on sentence index . Another issue with Firefox about triggering onEnd function at any.cancel()
method unlike chromium based browsers.This is different from tts-native: voices needs to be installed from OS so a modal will show up
TtsHelpModal
to guide users for each platform. Note: by default for windows for example you have what the keyboard layout have of languages also it may need a restart to see the newly installed language.Modified Modal.tsx to accept new props: icon and styling just for (backgroundColor, borderRadius)
Side effects
Testing
Observations
I tested this PR on linux and I got different result at each browser:
Chrome: works as expected + nice voices.
Firefox: I can't pause (linux) +
using opensource voice library frombetter sounded voices can be downloaded from pied (Note: you need to restart the browser after selecting a voice).mbrola mb-en1
sounds roboticBrave: (linux) not working at all because there is no build in speech to text like google chrome does.
Windows on the other hand.. most browsers can run good as long you download the voices from the OS it self.
I didn't test it on mac OS...
Resolved issues
Fixes: #2918