Page MenuHomePhabricator
Create Task
Maniphest T87116

reconsider whether Common.js should run in Special:ContentTranslation
Closed, DeclinedPublic
Actions

Description

Common.js runs in all pages. It assumes a lot of things: that a skin is fully loaded (see T86850), that the sidebar is visible, and so on. It applies changes to articles and common special pages, but ContentTranslation is a very different special page - practically no skin, no side bar, no usual elements, and so on. The only significant element that it has in common with other special pages is the top personal bar.

Common.js can run conditionally using

mw.config.get( 'wgCanonicalSpecialPageName' ) === 'ContentTranslation'

But maybe it's even better not to run it on ContentTranslation at all. There is a precedent for this already - MobileFrontend has its own JS file.

This issues already caused problems in es.wikipedia and eo.wikipedia, and may cause more issues in other projects where CX will be deployed in the future, so we should think about this soon.

Related Objects
Search...

Event Timeline

Amire80 created this task.Jan 17 2015, 10:51 PM
Amire80 raised the priority of this task from to Needs Triage.
Amire80 updated the task description. (Show Details)
Amire80 added a project: ContentTranslation.
Amire80 subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 17 2015, 10:51 PM
Amire80 added a comment.Jan 18 2015, 1:01 AM

... And there are issues in id.wikipedia, too.

Krenair subscribed.Jan 18 2015, 1:02 AM
Arrbee added a parent task: T87162: [Tracker] CX Phase 1 deployment issues.Jan 19 2015, 8:06 AM
• santhosh subscribed.Jan 22 2015, 2:55 AM
• santhosh added a comment.Feb 4 2015, 3:59 AM

I don't recommend this. Common.js is essential feature and it is better to support it. If we see some site having buggy Common.js, that need fix. In near future, we do have a plan to support translation tools by communities written in Common.js to extend CX translation tools.

• santhosh triaged this task as Low priority.Feb 4 2015, 4:00 AM
• MZMcBride subscribed.Feb 4 2015, 4:08 AM

Just tangentially, MobileFrontend is a pretty bad precedent for anything. It does all manner of crazy things that it shouldn't be doing.

Arrbee closed this task as Declined.Mar 24 2015, 9:02 AM
Arrbee claimed this task.
Ricordisamoa subscribed.Mar 24 2015, 11:00 AM

@Arrbee why? Adding links to the top personal bar is indeed a use case for the Common.js, and it could even extend CX as Santhosh wrote.
The only desktop special pages forbidding Common.js do so for security reasons AFAIK.

• santhosh added a comment.Mar 24 2015, 11:04 AM

@Ricordisamoa, By declined, @Arrbee meant "Common.js will continue to executee in Special:CX as it is doing now." :)
The bug was whether we need to disable it? Of course we wont disable it.

Ricordisamoa added a comment.Mar 24 2015, 11:28 AM
In T87116#1144671, @santhosh wrote:

@Ricordisamoa, By declined, @Arrbee meant "Common.js will continue to executee in Special:CX as it is doing now." :)
The bug was whether we need to disable it? Of course we wont disable it.

I'm sinking through the floor... :-<

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL