Page MenuHomePhabricator

Remove deprecated akeytt()
Closed, ResolvedPublic

Description

I think it is about time we truly removed akeytt() from wikibits.js. This function to update the tooltips and add keyboard shortcuts for them has been deprecated for a while. I have a patch that removes it, but I need to repair ajaxwatch.js which abuses this function.

Does anyone think we should provide a "dummy" function so that other scripts that are potentially using this function won't break ?


Version: unspecified
Severity: minor

Details

Reference
bz20847

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:51 PM
bzimport set Reference to bz20847.
bzimport added a subscriber: Unknown Object (MLST).
TheDJ created this task.Sep 28 2009, 12:28 PM

Created attachment 6593
Remove akeytt() function

ajaxwatch was using akeytt. However it was never updating the tooltip on wiki's that were not using the old ta Object to add tooltips and accesskeys. By now, that would be almost any wiki.

As such this updating of the tooltip functionality was broken on almost all wiki's and I consider it to be a separate issue if it needs fixing.

attachment akeytt.patch ignored as obsolete

brion added a comment.Sep 28 2009, 7:06 PM

Might be best to leave an empty stub function; that way any old scripts that call it will still run, just they won't get their tooltips added.

Created attachment 6597
Remove akeytt() function, but leave dummy

Same patch, but now leaving a dummy function in place.

Attached:

Applied in r57968. Thanks.