Page MenuHomePhabricator

Add missing colors for certain elements to pagelib theme transform
Closed, ResolvedPublic1 Estimated Story Points

Description

Note - Theme issues with Edit Previews are handled here https://phabricator.wikimedia.org/T174842
enwiki > Mycenaean Greek

on edit preview on first section

Screen Shot 2017-07-21 at 9.58.18 AM.png (1×862 px, 191 KB)

Non-edit preview presentations issues are listed below.
on non-edit preview presentation of the article if you un-collapse the first table

Screen Shot 2017-07-21 at 9.58.23 AM.png (1×862 px, 347 KB)

Screen Shot 2017-07-21 at 10.07.31 AM.png (1×862 px, 152 KB)

enwiki > John McCain

Screen Shot 2017-07-21 at 10.14.15 AM.png (1×862 px, 479 KB)

enwiki > Chris Cornell

Screen Shot 2017-07-21 at 10.16.23 AM.png (1×862 px, 278 KB)

Screen Shot 2017-07-21 at 10.18.00 AM.png (1×862 px, 196 KB)

enwiki > Audioslave

screenshot-2017-07-21-12-21-33-009749588.png (2×1 px, 530 KB)

screenshot-2017-07-21-12-20-47-242517449.png (2×1 px, 261 KB)

enwiki > Toast

Screen Shot 2017-07-21 at 10.20.04 AM.png (1×862 px, 165 KB)

enwiki > Mathematics

screenshot-2017-07-21-12-23-58-259666208.png (2×1 px, 222 KB)

enwiki > Manchester United F.C.

screenshot-2017-07-21-12-25-00-291763958.png (2×1 px, 372 KB)

screenshot-2017-07-21-12-25-13-769498059.png (2×1 px, 197 KB)

screenshot-2017-07-21-12-24-39-644428530.png (2×1 px, 259 KB)

enwiki > Gantz

Screenshot_1500861089.png (800×480 px, 63 KB)

Screenshot_1500861081.png (800×480 px, 231 KB)

enwiki > Kill 'Em All > Music and lyrics

device-2017-07-25-123520.png (2×1 px, 214 KB)

Event Timeline

If this is edit preview only, is this an issue with the integration or page library?

If this is edit preview only, is this an issue with the integration or page library?

pagelib theme transform when applied to edit preview html

@Niedzielski lemme know if there's a better way to phrase the ticket title.

@Niedzielski I may amend the description to include some other missing bits which may affect the regular article presentation (as opposed to the edit preview html).

Mhurd updated the task description. (Show Details)

@Niedzielski do you know of any other 'sore thumb' theming areas we could add to this ticket? Hoping to knock out as much low-hanging fruit as possible in one pass if we can...

Mhurd updated the task description. (Show Details)

I think there are some on the tables in https://fr.m.wikipedia.org/wiki/Liste_de_sondages_sur_l%27%C3%A9lection_pr%C3%A9sidentielle_fran%C3%A7aise_de_2017?oldid=138519598 and the infobox for French Obama. Visit a non-English wiki or a non-mainspace page and I'm sure you will be able to identify many more.

Mhurd updated the task description. (Show Details)

@JMinor I threw this on Goat, but lemme know if you want to wait. My preference would be to knock these out now. I don't think it would take very long - these are pretty easy to test :)

@Niedzielski

I think there are some on the tables in https://fr.m.wikipedia.org/wiki/Liste_de_sondages_sur_l%27%C3%A9lection_pr%C3%A9sidentielle_fran%C3%A7aise_de_2017?oldid=138519598 and the infobox for French Obama. Visit a non-English wiki or a non-mainspace page and I'm sure you will be able to identify many more.

Oh good idea!

Niedzielski added a subscriber: ABorbaWMF.

^^^ @ABorbaWMF, this would be a good opportunity for some further exploratory testing and bug reporting!

Per our standup, lets definitely tackle these (and any others we find) over the course of the beta.

Parking this in Needs Acceptance, so we can add to the list before we pull into development.

Mhurd updated the task description. (Show Details)

Per our meeting, there's enough change risk here that this should not be merged until 5.6.1

This PR has 79 commits over 12 days and is still marked WIP. 1 point can't be right! :]

@Niedzielski we don't point by complexity ;) The point was an attempt to use the point system for Kanban task limiters, but it didn't work.

Testing criteria:

  • Load the pages in the description in Dark Mode on 5.6.1
  • Verify that the areas identified in the screen shot that are white or light blue are now themed in dark colors as expected.