Page MenuHomePhabricator

RevisionSlider should not ignore uselang parameter
Closed, ResolvedPublic2 Estimated Story Points

Description

When uselang=xyz is specified in the URL RevisionSlider only considers it for the first diff shown. Once user selects other revisions custom language is ignored and diff for the language version selected in user preferences is loaded instead (this affects the langauage of reloaded parts of UI). Furthermore, new URLs generated by the extension do not contain uselang parameter.

See below how it looks like on wiki with default and user language set to English and URL initially containing uselang=he

revslider-uselang-bug.png (1×1 px, 119 KB)

  1. URL no longer includes uselang=he
  2. Diff headers are in English instead of Hebrew

Event Timeline

Change 302240 had a related patch set uploaded (by WMDE-leszek):
Do not ignore uselang paramter

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

WMDE-leszek set the point value for this task to 2.Aug 1 2016, 11:41 AM
WMDE-leszek moved this task from Proposed to Review on the TCB-Team-Sprint-2016-07-14 board.
WMDE-leszek updated the task description. (Show Details)

Change 302240 merged by jenkins-bot:
Consider custom user language setting, and include all parameters in generated URLs

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

Addshore subscribed.

Will be deployed this week.