Last modified: 2014-06-17 17:02:36 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 T36598, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 34598 - [SimpleSearch] Sometimes two search suggestion boxes are rendered instead of one
[SimpleSearch] Sometimes two search suggestion boxes are rendered instead of one
Status: RESOLVED FIXED
Product: MediaWiki skins
Classification: Unclassified
Vector (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-22 19:51 UTC by Erik Moeller
Modified: 2014-06-17 17:02 UTC (History)
6 users (show)

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


Attachments
Broken search box with additional search rendered underneath main search box (33.68 KB, image/png)
2012-02-22 19:51 UTC, Erik Moeller
Details
Correct search behavior (15.48 KB, image/png)
2012-02-22 19:52 UTC, Erik Moeller
Details
When the screen is resized, the whole secondary search box becomes visible. (39.74 KB, image/png)
2012-02-22 19:52 UTC, Erik Moeller
Details

Description Erik Moeller 2012-02-22 19:51:48 UTC
Created attachment 10070 [details]
Broken search box with additional search rendered underneath main search box

Tested on MediaWiki.org:

When typing in the search box, sometimes a second search box is rendered underneath or next to the primary search box. Both search boxes are active and respond to events, although the second search box exhibits legacy behavior (no bold highlight of letters typed, no "contains" search option).

Tested in Chrome 17 on Ubuntu 11.04.

Steps to reproduce: Repeatedly load any page on MediaWiki.org with a clear cache. Type in the search box. In a minority of cases, the broken behavior is present.
Comment 1 Erik Moeller 2012-02-22 19:52:10 UTC
Created attachment 10071 [details]
Correct search behavior
Comment 2 Erik Moeller 2012-02-22 19:52:35 UTC
Created attachment 10072 [details]
When the screen is resized, the whole secondary search box becomes visible.
Comment 3 Erik Moeller 2012-02-22 19:53:09 UTC
(Note that while the screenshots indicate that I'm logged in with gadgets active, I've been able to reproduce this issue logged out.)
Comment 4 Brion Vibber 2012-02-22 19:53:37 UTC
I think we've seen this before when both Vector's SimpleSearch and the other search option were enabled at once.... thought we resolved it in configuration though.
Comment 5 Brion Vibber 2012-02-22 21:00:20 UTC
Looks like live configuration has them both on, so either they need to be fixed to disable the one when the other's active or we should disable $wgEnableMWSuggest.

(But does the vector extension's simplesearch only work on Vector skin?)
Comment 6 Brion Vibber 2012-02-22 21:48:41 UTC
Vector's SimpleSearch attempts to disable MWSuggest (the built-in thingy)...

I suspect that sometimes they get loaded in the wrong order; if simpleSearch's ready function executes before mwsuggest.js gets run then it won't call the function to disable it.
Comment 7 Brion Vibber 2012-02-22 21:56:58 UTC
r112153 should fix this -- adds mediawiki.legacy.mwsuggest to ext.vector.simpleSearch's dependencies so we guarantee the load order between them.

Confirmed it fixes it on my local server, where it's very consistently loading wrong before the fix.

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


Navigation
Links