Page MenuHomePhabricator

Remove 100% width on mobile page-issue
Closed, ResolvedPublic3 Story Points

Description

the new page-issues style on mobile shouldn't be given a 100% width, because section issues might get weird when they're next to an infobox (on tablet widths anyway).

badbetterVector

Fix: remove line #30 in ambox.less

QA steps

  • Click the issues to open the issues overlay and check they display as expected in the overlay.

Details

Related Gerrit Patches:
mediawiki/skins/MinervaNeue : masterOn tablet, issues boxes shouldn't take up full screen

Event Timeline

Restricted Application changed the subtype of this task from "Deadline" to "Task". · View Herald TranscriptAug 22 2018, 11:18 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Jdlrobson set the point value for this task to 2.Aug 22 2018, 3:46 PM
Jdlrobson changed the point value for this task from 2 to 3.Aug 22 2018, 3:55 PM
Jdlrobson added a subscriber: Jdlrobson.

Estimation bumped from 2 to 3 as there is a little risk here given ambox templates are usually tables and table CSS rules might come into conflict here.

Jdlrobson moved this task from To Do to Doing on the Readers-Web-Kanbanana-Board-Old board.
Jdlrobson updated the task description. (Show Details)Aug 22 2018, 9:53 PM

Change 454704 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] On tablet, issues boxes shouldn't take up full screen

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

Jdlrobson added a subscriber: jan.

Looks like @jan was right, this did prove to be an easy one unless I'm overlooking something.

Looks good! I think the QA here is pretty simple, so I tested this on several devices in browser stack.
Tested on iPhone, iPad, Google Nexus in Chrome and Firefox.

iPhone XiPadNexus 7Nexus 9

As seen on the Nexus 9 example, even thought the issue text is very short, the issue still stretched to fill the available space, while letting the infobox sit beside it. So LGTM! I don't think this needs further QA. I can pass it on the @alexhollender for sign-off.

Change 454704 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] On tablet, issues boxes shouldn't take up full screen

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

alexhollender added a project: Product-QA.

Looks good.

United States ArmyThe AnimatrixEri Kitamura

Looks good to me for the first two articles on all three resolutions. The display of the issues overlay also looks fine. No issues with other tables on those articles.

I don't see any page issues for the article "United_States_Army" though! So could not verify it for that article. Other than that, all good from my side.

ovasileva closed this task as Resolved.Aug 28 2018, 11:02 AM

@Ryasmeen - perhaps you weren't in the correct bucket? I was able to see them:

Looks good! I'm going to go ahead and resolve this.