Page MenuHomePhabricator

Link Preview version 2 is broken on tablet in beta, current master dev branch
Closed, ResolvedPublic

Description

While trying out WP Beta on my Nexus 9 (5.1.1), I noticed that link preview version 2 is broken. It doesn't come up all the way, and text is cut off in portrait (no text at all in landscape). This is the case currently in the master dev branch as well.

It's fine on my phone (in both portrait and landscape).

Version 1 is fine on both devices, in both orientations. Only Version 2 is affected.

Event Timeline

Mholloway claimed this task.
Mholloway raised the priority of this task from to Unbreak Now!.
Mholloway updated the task description. (Show Details)
Mholloway moved this task to To Do on the Mobile-App-Sprint-58-Android board.

Screenshots from current Wikipedia Beta

Screenshot_2015-05-28-12-47-33.png (2×1 px, 789 KB)

Screenshot_2015-05-28-12-47-42.png (1×2 px, 782 KB)

Change 214417 had a related patch set uploaded (by Mholloway):
Fix margins on link preview dialog 2 (display correctly on tablets)

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

Caused by a bad margin setting in dialog_link_preview_2.xml. This has probably never displayed correctly on tablets. Fixed now. Still works on phones.

Screenshot_2015-05-28-15-29-06.png (2×1 px, 852 KB)

Screenshot_2015-05-28-15-29-13.png (1×2 px, 856 KB)

Change 214417 merged by jenkins-bot:
Fix margins on link preview dialog 2 (display correctly on tablets)

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