Page MenuHomePhabricator

bug - Lead image should not have margins on screens less than 893
Closed, ResolvedPublic

Description

On screens less than 893, content should not match the lead image, but have 55 px margins. Lead images will stretch full width, edge-to-edge on iPads in both portrait and landscape.
Original task T91743

Current:

Screenshot_2015-05-06-13-55-52.png (2×1 px, 865 KB)

Design:

IMG_8373.jpg (1×768 px, 105 KB)

Event Timeline

KHammerstein raised the priority of this task from to Needs Triage.
KHammerstein updated the task description. (Show Details)
KHammerstein subscribed.
KHammerstein renamed this task from Lead image should not have margins on screens less than 893 to bug - Lead image should not have margins on screens less than 893.May 8 2015, 4:15 PM
KHammerstein moved this task from Incoming to 2015-16 Q3 on the Web-Team-Backlog board.
KHammerstein set Security to None.

Where does 893px come from?
Note we define the threshold for mobile as 320px and then tablet as 768px
We should introduce any new numbers.

@Jdlrobson
768px is portrait width, 893px is landscape width (not including margins)
893 also works as a good max width for desktop.
We've been using this for awhile.

We've never used 898px before so the team should take the time to consider whether this is the most sensible threshold (e.g. landscape on what? ios? android? blackberry?) rather than rushing into using this number. A short spike should solve that. I'll setup a task.

@Jdlrobson 768 makes sense as the entry threshold to tablet, but we need a max-width also. iPad landscape is 1024, and with 65px margins on each side you get 894. (Sorry not 893)

I thought it worked pretty well to constrain to 894 on desktop too for consistency, but I think its right to explore desktop more thoroughly.
Thanks Jon

@Jdlrobson Just to clarify this is mainly to solve the bug that lead images are not full-width on tablets.

Current version of WikidataPageBanner extension already displays full width banner images on all screen sizes -

Screenshot from 2015-12-04 17-07-34.png (586×809 px, 202 KB)

Does that mean the body content would have to accomodate more margin to account for 65px margin on both sides?

Sumit assigned this task to KHammerstein.