Skip to content

Fix unbind click event#621

Merged
joedolson merged 58 commits intomainfrom
fix-unbind-click-event
Jun 15, 2025
Merged

Fix unbind click event#621
joedolson merged 58 commits intomainfrom
fix-unbind-click-event

Conversation

@joedolson
Copy link
Member

This is an update to PR 575, , attempting to identify the functional changes in that PR.

@jeanem If you have a chance, can you confirm whether I caught everything you'd changed in the previous PR? The number of formatting changes make it difficult to be sure.

terrill and others added 30 commits November 11, 2022 13:56
Issues with speech were discovered in Debian 11.6 (Bullseye) with the
[espeak](https://espeak.sourceforge.net/) package.

* Firefox and [Chromium](https://www.chromium.org/Home/) do not respect
  the volume parameter; the configured phrase "Hi!" is spoken for every
  instances of AblePlayer.
* Google Chrome will lock speech synthesis for the browser if the rate
  parameter is changed. Speech synthesis will be locked until the
  affected tab is closed.

Browsers are consistently silent when SpeechSynthesis initialized with
`\x20` (space). Silent initialization allows for the removal of volume
and rate parameters, improving the likelyhood of universal speech
support.

Changes were tested against all listed AblePlayer browsers except for
Windows.
Added a screenshot of the player
jeanem and others added 23 commits March 21, 2025 08:28
This is only used once. If there are more uses, might make a function.

See #612
In 443f104, fullscreen dynamically set caption font size to a percentage of `vw`. At mobile sizes, however, those are minuscule. Check screen size before determining what size to set, and use a fixed size on any screen that will result in a caption size less than 12px at 100%.

Fixes #602
@joedolson joedolson requested a review from jeanem May 25, 2025 19:24
Copy link
Contributor

@jeanem jeanem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joedolson - I confirmed that the original code from the #600 PR for fixing the unbind event issue is present in this PR. Also I pulled this PR locally and tested this particular fix and looks like it is good to go. Thanks!

@joedolson
Copy link
Member Author

Thanks, @jeanem!

@joedolson joedolson merged commit a474593 into main Jun 15, 2025
1 check passed
@joedolson joedolson deleted the fix-unbind-click-event branch June 15, 2025 22:29
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.

8 participants