-
Notifications
You must be signed in to change notification settings - Fork 34
fix: resumeInfinity/onresume interaction #544
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
Open
tuncayuk
wants to merge
31
commits into
lightning-js:dev
Choose a base branch
from
tuncayuk:fix/announcer-issue
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
3170c73
fix: resumeInfinity/onresume interaction
tuncayuk 15614f1
chore: ensure target is a SpeechSynthesisUtterance in resumeInfinity …
tuncayuk f35ab5e
fix: ensure infinityTimer is checked against null in clear and resume…
tuncayuk 910df0c
fix: ensure resumeFromKeepAlive is strictly checked for true in onres…
tuncayuk ecb2ddc
fix: improve utterance management and error handling in speech synthesis
tuncayuk d43ee72
chore: add placeholder for getVoices method in speech synthesis
tuncayuk 3e56c9d
chore: improve error handling in speak function of speech synthesis
tuncayuk d2721b6
chore: enhance queue management and debounce handling in announcer
tuncayuk 6aa4a19
fix: add optional chaining to safely check timer state in resumeInfin…
tuncayuk a1ebc94
fix: enhance stop and clear functions to ensure clean state in speech…
tuncayuk 9910a08
fix: add double-check for utterance existence before resuming in resu…
tuncayuk 624604f
fix: add early return in clear function to handle non-existent state
tuncayuk fa88731
chore: remove unused pause event handling in speak function
tuncayuk 4fb38de
fix: add cancelPrevious option to AnnouncerUtterance for managing spe…
tuncayuk cc30b5e
chore: simplify stop api
tuncayuk 3a1a66e
fix: increase debounce duration in processQueue to improve performance
tuncayuk 7d842ee
fix: add waitForSynthReady function to ensure speech synthesis engine…
tuncayuk 41aa289
fix: add enableUtteranceKeepAlive option to improve speech synthesis …
tuncayuk 6f3a1ae
fix: remove previous resolve function in clear method
tuncayuk 2c5255f
fix: remove redundant utterances.delete calls in clear and startKeepA…
tuncayuk 8e9d109
fix: log result of speaking in processQueue for better debugging
tuncayuk 4dc14df
fix: include result in onend callback of speak function for better ha…
tuncayuk 8441944
Removed unneeded variable assignments.
michielvandergeest 9959d92
Removed debug log.
michielvandergeest 1d4884e
Removed debug log and renamed variable.
michielvandergeest 6e128c3
fix: update clear function to accept cancelPrevious option for better…
tuncayuk 0be466c
fix: replace optional chaining with explicit null checks for timer in…
tuncayuk eea2d60
fix: remove invalid utterance check in startKeepAlive function
tuncayuk ab06ead
fix: ensure initialized is set to false when speech synthesis API is …
tuncayuk 3cc5d28
fix: change log level from warn to debug for speech synthesis readiness
tuncayuk 560067e
fix: refactor enableUtteranceKeepAlive option to use a constant for b…
tuncayuk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.