jquery.suggestions should obey maxRows setting
Closed, ResolvedPublic

Description

Author: john

Description:
jQuery.suggestions has a maxRows property however it is never used and therefore has no effect.

Details

Reference
bz37316
bzimport raised the priority of this task from to Lowest.
bzimport set Reference to bz37316.
bzimport added a subscriber: Unknown Object (MLST).
bzimport created this task.Jun 2 2012, 10:21 PM

Note that the jquery.suggestions plugin (afaik) is no longer maintained and was temporarily developed by the Usability Initiative when jquery.ui.autocomplete had issues. This is no longer the case and most if not all scripts are now using jquery.ui.autocomplete. And when not already, they should be changed to use that so we can remove this old module.

matmarex added a subscriber: matmarex.

jquery.suggestions is still used, most notably for the search suggestions dropdown in all skins. There was some other module that was killed off a long time ago, you must have meant that.

The "default" for the unused option is currently 7… I'd change this to 10 if this bug is fixed, to reflect what the actual limit for the search suggestions is right now. (Or just fix this in that particular instance.)

Change 182642 had a related patch set uploaded (by Unicodesnowman):
Make jquery.suggestions obey maxRows setting

https://gerrit.wikimedia.org/r/182642

Patch-For-Review

Krinkle updated the task description. (Show Details)Jan 5 2015, 1:19 AM
Krinkle raised the priority of this task from Lowest to Normal.
Krinkle set Security to None.
Krinkle removed a subscriber: Unknown Object (MLST).

Change 182642 merged by jenkins-bot:
Make jquery.suggestions obey maxRows setting

https://gerrit.wikimedia.org/r/182642

matmarex closed this task as Resolved.Jan 5 2015, 2:32 PM
matmarex assigned this task to Unicodesnowman.
matmarex removed a project: Patch-For-Review.