Page MenuHomePhabricator

Migrate Special:CXStats to contenttranslation skin
Closed, ResolvedPublic

Description

In T263093: Create a custom skin for Content Translation special pages, we created a new skin named 'contenttranslation' to use CX Special pages. It is now used with the section translation variant.

This ticket is to migrate Special:CXStats to the new skin and avoid fragile special page override

Event Timeline

Change 666843 had a related patch set uploaded (by Santhosh; owner: Santhosh):
[mediawiki/extensions/ContentTranslation@master] Special:CXStats: Use contenttranslation skin

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

santhosh triaged this task as Medium priority.Feb 25 2021, 9:20 AM

I chose this ticket before T267103: Migrate Special:CX to contenttranslation skin so that we can test this low traffic page thoroughly before going to high traffic Special:CX

Setting medium priority since this is a tech debt and can break any time because of ongoing refactoring in Vector front.

Here is a screenshot based on the patch from my development environment. The cx specific skin is based on the SX design and hence the small visual changes. Since the SX's modern codebase will be eventually used for replacing CX's old codebase, @Pginer-WMF can confirm if this is ok.

BeforeAfter
image.png (876×1 px, 55 KB)
image.png (886×1 px, 57 KB)

@Pginer-WMF can confirm if this is ok.

Makes sense to me. There are some mall visual aspects to polish, but I guess that as we improve them as part of the Section Translation dashboard work, these will get fixed here too.

Change 666843 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Special:CXStats: Use contenttranslation skin

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

Jpita subscribed.

@santhosh it's kind of broken on mobile, but that's expected I guess?

image.png (998×1 px, 99 KB)

Yeah, this is expected. We had made the page reasonably responsive, but there are several quirks not addressed since 2015

  NODES
HOME 1
Note 3
os 21
visual 2