Page MenuHomePhabricator

Make the limited width and font size controls work for logged in users
Open, LowPublic5 Estimated Story Points

Description

Currently toggling limited width or font size throws a JavaScript error. We need to fix this before shipping to beta cluster. Ideally we'd like the solution to be a long-term and stable fix that we do not need to revisit again.

TODO

  • Talk with platform team about long term solutions in T345664.
  • Create a short term fix with a FIXME linking to the long term fix.
  • Add a test that means CI will break if the feature ever breaks again

Event Timeline

Jdlrobson triaged this task as High priority.
Jdlrobson created this task.
ovasileva set the point value for this task to 5.Thu, Nov 16, 6:39 PM

Change 975919 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Vector@master] Make the limited width and font size controls work for logged in users

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

Change 975919 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Make the limited width and font size controls work for logged in users

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

Jdlrobson lowered the priority of this task from High to Medium.Tue, Nov 21, 5:39 PM
Jdlrobson lowered the priority of this task from Medium to Low.
Jdlrobson lowered the priority of this task from Low to Lowest.Tue, Nov 21, 10:14 PM
Jdlrobson raised the priority of this task from Lowest to Low.Mon, Nov 27, 6:30 PM
Jdlrobson updated the task description. (Show Details)

We put together a document describing handover of the API to the mediawiki platform team and concerns with the existing code. The conversation will continue there outside the scope of this ticket.