Last modified: 2012-12-16 17:43:09 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 14444 - Ajax search feature does not work in modern skin
Ajax search feature does not work in modern skin
Status: RESOLVED INVALID
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.12.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Brion Vibber
:
Depends on:
Blocks: ajax
  Show dependency treegraph
 
Reported: 2008-06-06 16:23 UTC by Edward Rudd
Modified: 2012-12-16 17:43 UTC (History)
2 users (show)

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


Attachments

Description Edward Rudd 2008-06-06 16:23:32 UTC
The ajax search feature works fine using the monobook skin, but it fails to function properly using the modern skin.

The cause is the ajaxsearch.js code expects to find a div with the id of "content" and in modern the div is named "mw_content" instead.

This can be fixed either by one of the following
1. changing the modern theme to use the same ID as the rest of the themes..
2. change the rest of the themes to use mw_content id and update the ajaxsearch.js to use that ID as well
3. update ajax search to check BOTH ids.  
   body = document.getElementById("content") || document.getElementById("mw_content");
Comment 1 Brion Vibber 2008-06-06 16:33:18 UTC
The old ajax search is obsoleted by the new ajax suggest search; I'm probably going to remove it today, so this bug'll be obsolete. :)
Comment 2 Siebrand Mazeland 2008-08-16 22:08:56 UTC
Brion, has this been done? If so, please close.
Comment 3 Brion Vibber 2008-08-20 18:13:21 UTC
Now that 1.13 release is past, I've gone ahead and removed the old AJAX search in r39717.

The new suggestion search works fine in Modern, so mission accomplished. :)

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


Navigation
Links