Skip to content

Conversation

david419kr
Copy link

Description

Add a new configuration option "Pause/play on double click" that allows users to trigger pause/play functionality with double click instead of single click.

image

When this option is enabled:

  • Double click pauses/plays the video instead of single click
  • "Prevent pause/play from triggering on double click" is automatically disabled to prevent conflicts
  • "Disable fullscreen toggle on double click" is automatically enabled to prevent fullscreen conflicts

Related issue

Issue #71

Motivation and context

As I migrated from Potplayer, I wanted this feature to pause with double click, and then realized there is an open issue of this feature.

How has this been tested?

Tested on Windows 11 64bit, VLC 3.0 environment.

Checklist

  • I have tested these changes locally, and this fixes the bug/the new feature behaves as the expectation
  • The saved settings from the old version are compatible with the settings of the new version after this change
  • These changes only fix a single bug/introduces a single feature (Otherwise, open multiple Pull Requests instead, unless these bugs/features are closely related)
  • The commit messages are clear and detailed and follow the 50/72 rule (Otherwise use git rebase -i to modify the commit messages)
  • This Pull Request is ready to be reviewed, it's not a work in progress (It's perfectly okay to have WIP Pull Requests, just please indicate them as such by leaving the checkbox unchecked)

Add a new configuration option "Pause/play on double click" that allows users to trigger pause/play functionality with double click instead of single click.

When this option is enabled:
- Double click pauses/plays the video instead of single click
- "Prevent pause/play from triggering on double click" is automatically disabled to prevent conflicts
- "Disable fullscreen toggle on double click" is automatically enabled to prevent fullscreen conflicts
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.

1 participant