Page MenuHomePhabricator

[BUG] mobile-html article body has wrong background color
Open, HighPublicBug

Description

How many times were you able to reproduce it?

5/5

Steps to reproduce

  1. Go to https://en.wikipedia.org/api/rest_v1/page/mobile-html/Dog in the browser
  2. Open console
  3. Paste pagelib.ThemeTransform.setTheme(document, pagelib.ThemeTransform.THEME.BLACK) in the console to change theme to black

Expected results

Body is black

Actual results

Body is white

Context

It appears that this has to do with the recent fix for T214728 that unsets .content background color.
In mobile-html pages content applies to body so setting unset or initial on content overrides background from pagelib_theme_black_body. Turning off those 2 content rules fixes it.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 7 2019, 3:02 PM
Jhernandez added subscribers: bearND, Jhernandez.

@bearND to move under the appropriate epic and prioritise.

bearND triaged this task as High priority.Mar 13 2019, 6:55 PM

@Jhernandez Should we make T177433 an Epic?

bearND added a subscriber: Mhurd.Thu, Apr 11, 10:39 PM

@Mhurd Uploaded https://github.com/wikimedia/wikimedia-page-library/pull/182. I think we could get rid of the added rules for T214728, now that the base CSS endpoint is more stable and doesn't include most of the extraneous CSS rules from Minerva (see T220045).

@bearND oh cool! will check it out!

bearND claimed this task.Thu, Apr 11, 10:40 PM