Last modified: 2013-08-20 09:27:21 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 T55041, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53041 - Cannot click links or submit forms on special pages with an input box
Cannot click links or submit forms on special pages with an input box
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UniversalLanguageSelector (Other open bugs)
master
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-19 14:50 UTC by Aude
Modified: 2013-08-20 09:27 UTC (History)
9 users (show)

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


Attachments

Description Aude 2013-08-19 14:50:24 UTC
With master of ULS (specifically with https://gerrit.wikimedia.org/r/#/c/79755/), on special pages with an input box, I cannot click on any links or submit forms after clicking inside the input box.

(for example, Special:Allpages)
Comment 1 Amir E. Aharoni 2013-08-19 14:59:13 UTC
Hmm. Special:AllPages works for me on my wiki.
Comment 2 Amir E. Aharoni 2013-08-19 15:13:50 UTC
aude	if it matters at all, i'm in chrome now
aharoni	aude, Nikerabbit - true - it breaks in chrome
aude	ok, browser specific
aude	think i found the line of code that might be a problem
aude	if i comment out $( 'html' ).click( $.proxy( this.hide, this ) ); in jquery.ime.js
aude	then it seems to work
aude	don't know what that is supposed to do
aude	well, other than what the code comment says
Comment 3 Aude 2013-08-19 15:23:33 UTC
issue also applies to the search box, so can occur on any page
Comment 4 Bartosz Dziewoński 2013-08-19 15:28:13 UTC
That's because #hide returns false, preventing all other click handlers, including browser's native ones, from running…
Comment 5 Bartosz Dziewoński 2013-08-19 15:33:22 UTC
I'd submit a patch, but it's a really annoying process for ULS.

Someone should probably just remove that "return false;", in this function and maybe a bunch of others, since they don't seem to need it either.

Or alternatively, replace that line with $( 'html' ).click( function () { imeselector.hide() } ); to gobble up the return value and be more careful in the future.
Comment 6 Bartosz Dziewoński 2013-08-19 15:57:23 UTC
I submitted one after all. https://github.com/wikimedia/jquery.ime/pull/261
Comment 7 Gerrit Notification Bot 2013-08-19 16:34:17 UTC
Change 79810 had a related patch set uploaded by Amire80:
Update jquery.ime from upstream

https://gerrit.wikimedia.org/r/79810
Comment 8 Gerrit Notification Bot 2013-08-20 07:22:37 UTC
Change 79810 merged by jenkins-bot:
Update jquery.ime from upstream

https://gerrit.wikimedia.org/r/79810
Comment 9 Aude 2013-08-20 09:27:21 UTC
Thanks Bartosz and Amir! the issue is fixed now.

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


Navigation
Links