Last modified: 2010-01-27 15:31:06 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T23850, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 21850 - r59935 interferes with live event binding
r59935 interferes with live event binding
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UsabilityInitiative (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Trevor Parscal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-12-14 18:45 UTC by Alex Z.
Modified: 2010-01-27 15:31 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Test code (1.22 KB, application/x-javascript)
2009-12-14 18:45 UTC, Alex Z.
Details

Description Alex Z. 2009-12-14 18:45:33 UTC
Created attachment 6881 [details]
Test code

Something in r59935 is screwing with binding events to elements with live(), (I've only tested the click event though). I've narrowed it down fairly definitively to r59935, r59934 works as expected. For live-bound click events on single elements, it causes them to run when clicking anywhere on the page, even if the element isn't visible. This behavior is also evident on HEAD (r60039 as of now)

I've attached some short test code that can be added as a user script. It adds a section to the toolbar, which adds a single dialog. Clicking the "click" link in the dialog should trigger an alert message, but as of r59935, it triggers when clicking anywhere outside the textarea, even if the dialog isn't open.
Comment 1 Alex Z. 2009-12-14 19:10:26 UTC
Forgot to mention, I was testing on Firefox 3.5.5 on Fedora 11
Comment 2 Roan Kattouw 2010-01-27 15:31:06 UTC
r59935 added the inherit plugin, which presumably broke .live(), but it was removed in r61499

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links