Page MenuHomePhabricator

Minerva skin HTML should be consistent with Vector skin HTML
Closed, ResolvedPublic


In mobile skin:
#content_wrapper should be changed to #content
and #content to #bodyContent

Existing uses of the #content element should be evaluated carefully
$ ag -a '#content' resources/

Do not break cached page HTML!

Event Timeline

Change 229937 had a related patch set uploaded (by Jdlrobson):
Make Minerva skin more consistent with Vector

Jhernandez added a subscriber: bmansurov.

Verified in Beta cluster.

@Jdlrobson @bmansurov I'm not sure but it seems like the patch (based on the commit message) is doing more things than what the description of this task specifies. This is rolling out tomorrow, please verify anything other than #content and #bodyContent on Beta Cluster today!

If there's more stuff changed please updating the phab description would be useful too.

I did some testing on labs and I didn't find any issues.

There should be no fallback. The additional changes were just weakening some overly specific existing css rules.