Page MenuHomePhabricator

Excessive margin in notifications overlay on tablets
Closed, ResolvedPublic


Screen Shot 2015-06-18 at 3.11.40 PM.png (545×497 px, 30 KB)

Note the excessive whitespace.
We should get rid of the left margin to remove the gap.

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added subscribers: phuedx, JKatzWMF, dr0ptp4kt and 3 others.

I've been poking at this a bit. Not too happy with any of the solutions so far :-/
Sorry can't tag it right now as on mobile device and gerrit is not mobile friendly

Change 219989 had a related patch set uploaded (by Phuedx):
Fix the positioning of overlay headings

Talk looks fine to me in stable and beta - the close icon and content are aligned:

Screen Shot 2015-06-25 at 1.14.07 PM.png (420×929 px, 21 KB)

@bmansurov was concerned about a style regression but I honestly can't see it and am not sure we should block on it: =>

It's not only a minor styling issue, it's also about doing the right thing. You're basically ignoring "maxwidth - 2*paddings" and displaying content differently.

To quote @Jdlrobson yourself:

When focusing search or editor, the close icon should match up with the content area and the hamburger icon. [1]

You've written this patch to fix the spacing issue, but ignoring the new spacing issue.


Change 221090 had a related patch set uploaded (by Bmansurov):
Remove wikitext editor padding in alpha and beta.

Change 221090 merged by jenkins-bot:
Fix the positioning of overlay headings and content

Change 219989 abandoned by Jdlrobson:
Fix the positioning of overlay headings