- Issue created by @odlein
- ๐ฎ๐ณIndia Shreya_98
Shreya_th โ made their first commit to this issueโs fork.
- @shreya_th opened merge request.
- ๐บ๐ธUnited States theMusician
Thank you for the report and patch.
I believe this is related to the Able Player project JS itself, https://github.com/ableplayer/ableplayer/issues/581. I wonder if their proposed fix would also resolve it here until the core project releases a new update? - ๐บ๐ธUnited States theMusician
I tried to reproduce the issue and have not succeeded. Is this for a remote video or local video? Can you share thee exact settings you tried to save that did not take effect until refreshing the page?
Thanks
The issue is present in 3.0.0-beta1 and 3.x-dev, and does seem to be the same typo mentioned in the core issue you linked: https://github.com/ableplayer/ableplayer/issues/581#issuecomment-1648586370
I was using a remote video (YouTube), and haven't tried with a local video. It is affecting all caption preferences (background, text color, etc).- ๐บ๐ธUnited States theMusician
Thank you odelin. I'll try to get your patch in place tomorrow and update dev. Long-term I worry about patching the JS directly as when Able Player itself updates, we'll need to verify the patch is no longer needed or still needed and reroll.
I wonder if I can add the patch via composer with the module?
Thank you again for reporting this and supplying a fix.
Max
- ๐บ๐ธUnited States theMusician
Adding the code commit MR as a patch file to play nice with a composer workflow. This will allow the module to maintain synchronicity with the Able Player parent project while allowing for patching of the parent project JS in between releases.
- ๐บ๐ธUnited States theMusician
The composer route did not pan out. Merging.
- Status changed to Needs review
about 1 year ago 6:37pm 15 September 2023 - Status changed to RTBC
about 1 year ago 10:08am 16 September 2023 - ๐ฎ๐ณIndia shubham_jain
Hi everyone, I reviewed it and it is working fine. Moving to RTBC.
- Status changed to Fixed
about 1 year ago 5:09pm 22 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.