Page MenuHomePhabricator

Remove deprecated jquery.ui from PageTriage
Open, Needs TriagePublic

Description

  • replace .button() in 39 spots
  • replace .draggable() in 1 spot
  • turn off jquery.ui in PageTriage in a localhost environment and see if anything else breaks. identify and fix breakages

Motivation: jquery.ui is maintained upstream, but MediaWiki has marked it as deprecated since 2017 because they want to reduce the amount of libraries/bandwidth being loaded on the front end

Converting PageTriage to Codex (T208256) is a big project that would probably change the front end a lot, so may not have consensus and the work may not happen. So this ticket is an intermediate step. If there's a way to get rid of jquery.ui without changing the front end appearance of PageTriage significantly and without creating a lot of work, we should move forward with that. Perhaps we can use a .button() from one of our supported libraries.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Note that removing jquery.ui from Pagetriage will not remove jquery.ui from page load since WikiLove also uses it and is closely coupled with the code here.

I wanted to note that the new toolbar ui (currently behind the pagetriage_tb url parameter) replaces draggable() already. You can access it by following review links from the feed with the parameter set: https://en.wikipedia.org/wiki/Special:NewPagesFeed?pagetriage_tb=new. It's a partial vue migration that should work as a drop in replacement for the current one.
see T340117 for more info

I would suggest that we could go ahead and promote the new toolbar as it is and then remove the old toolbar if everyone is happy with it. Migration of individual buttons could then be handled behind their own url parameters as they get done over time.