Skip to content

Conversation

CyanEngineer
Copy link

I think it would be really nice to be able to select and copy the caption text. The caption container has pointer-events disabled as it would otherwise eat the mouse input for the whole player area. But re-enabling pointer-events for just the text container makes the text selectable without affecting the player outside of the area with text.

Copy link
Member

@SamantazFox SamantazFox left a comment

Choose a reason for hiding this comment

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

Sounds good to me!

@SamantazFox SamantazFox added the need-testing This feature needs to be deployed and tested to see if it's working, and doesn't break something label Jun 21, 2025
@syeopite syeopite removed the need-testing This feature needs to be deployed and tested to see if it's working, and doesn't break something label Aug 23, 2025
@syeopite
Copy link
Member

Tested and works as expected.


Though this isn't a problem right now I'm concerned that this feature could conflict with implementing freely movable captions like those seen on YouTube's official player. You can't really differentiate whether the user wants to move the captions box or select the text since it involves the same mouse movements over a very tiny space that overlaps.

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.

3 participants