Skip to content
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

Remove dependency on tts #3651

Merged
merged 2 commits into from
Nov 28, 2023
Merged

Remove dependency on tts #3651

merged 2 commits into from
Nov 28, 2023

Conversation

emilk
Copy link
Owner

@emilk emilk commented Nov 28, 2023

We were using tts for the web-only screen reader. This was overkill, to say the least. It is now replaced with ten lines of web-sys calls.

@emilk emilk added eframe Relates to epi and eframe dependencies Pull requests that update a dependency file labels Nov 28, 2023
@emilk emilk changed the title Replace use of tts with simple web-sys calls Remove dependency on tts Nov 28, 2023
@emilk emilk merged commit a1f3c71 into master Nov 28, 2023
35 checks passed
@emilk emilk deleted the emilk/remove-tts branch November 28, 2023 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file eframe Relates to epi and eframe
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant