Skip to content

Comments

Sync text of the alerts on screen with voice timing#2280

Merged
rafaellehmkuhl merged 2 commits intobluerobotics:masterfrom
rafaellehmkuhl:keep-alerts-min-time-on-screen
Dec 8, 2025
Merged

Sync text of the alerts on screen with voice timing#2280
rafaellehmkuhl merged 2 commits intobluerobotics:masterfrom
rafaellehmkuhl:keep-alerts-min-time-on-screen

Conversation

@rafaellehmkuhl
Copy link
Member

Now the user will see on the screen exactly the same text that is being spoken.

If the user has the voice disabled, it will use the same timing, since I've modified the logic to mute (volume = 0) the speaking instead of actually disabling it.

Watch the video with the audio on:

Cockpit.0.0.0.2025-12-08.13.25.07.mp4

Fix #2279

This way we can use the `onend` event to keep the alert text in the screen for a reasonable amount of time.
@rafaellehmkuhl rafaellehmkuhl merged commit d607f10 into bluerobotics:master Dec 8, 2025
10 of 11 checks passed
@rafaellehmkuhl rafaellehmkuhl deleted the keep-alerts-min-time-on-screen branch December 8, 2025 18:32
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.

Alerts should stay in the screen for a minimum amount of time

2 participants