Skip to content
This repository was archived by the owner on Sep 28, 2025. It is now read-only.

Conversation

EternalPatience
Copy link

Summary

This pull request introduces two new custom events, CONTAINER_TAP and CONTAINER_DBLTAP, and updates CONTAINER_CLICK and CONTAINER_DBLCLICK to be triggered alongside their respective original events. The new custom events have potential utility for distinguishing player behavior on mouse versus touch events. For instance, a CONTAINER_TAPevent could be used to display media controls on the first tap and trigger stop/play on the second.

Furthermore, this PR includes an additional argument for the trigger touch and click events of the container, which provides the original event. This enhancement could prove valuable for scenarios where HTML elements have already been created by other plugins and nested in a Container, and differentiation is required between the triggered events.

Changes

  • src/base/events:
    • events.js
  • src/components/container:
    • container.js
    • container.test.js
  • src/utils/:
    • utils.js
    • utils.test.js

How to test

Try binding CONTAINER_TAP/CONTAINER_DBLTAP/CONTAINER_CLICK/CONTAINER_DBLCLICK with listenTo

* Trigger CONTAINER_CLICK/DBLCLICK/TAP/DBLTAP with the original event.
This can be useful in case the container already contains custom
created by other plugins, and it is necessary to determine
target of the event.
* Rework `tap delay` mechanism and remove DoubleEventHandler
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant