Skip to content

Fix one() for array events on target objects#9189

Open
puneetdixit200 wants to merge 1 commit into
videojs:mainfrom
puneetdixit200:fix/9183-one-array-events
Open

Fix one() for array events on target objects#9189
puneetdixit200 wants to merge 1 commit into
videojs:mainfrom
puneetdixit200:fix/9183-one-array-events

Conversation

@puneetdixit200

Copy link
Copy Markdown

Summary

  • register a separate one-time wrapper for each cross-target event type
  • keep off-by-original-listener support by preserving the listener guid
  • update the regression test for one() with an array of event types

Fixes #9183

Tests

  • npm run lint-errors
  • CHROME_BIN='/Applications/Google Chrome.app/Contents/MacOS/Google Chrome' ./node_modules/.bin/karma start test/karma.conf.js --browsers ChromeHeadless --single-run --client.qunit.filter='one() can add a listener to an array of event types on a different target object'
  • CHROME_BIN='/Applications/Google Chrome.app/Contents/MacOS/Google Chrome' ./node_modules/.bin/karma start test/karma.conf.js --browsers ChromeHeadless --single-run --client.qunit.filter='mixins: evented'
  • npm run build:js

Signed-off-by: Deepak kudi <deepakkudi23@adsl-172-10-9-116.dsl.sndg02.sbcglobal.net>
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.

Bug: one() with multiple event types removes all listeners on first trigger

1 participant