Page MenuHomePhabricator

Top margin issues on enwiki main page
Closed, ResolvedPublic

Description

Bug filed via OTRS: https://ticket.wikimedia.org/otrs/index.pl?Action=AgentTicketZoom;TicketID=9948597

Steps to reproduce

Tap on the "Today's featured page" widget to open the Main page in the app.

Expected

Main page opens as a new article tab showing page content is shown starting from just below the app bar, with "Today's featured article" as the first heading on the screen.

Actual

Top of the screen is obscured behind the app bar:

Screenshot_20170515-125408.png (1×1 px, 450 KB)

Note: when a new article tab is opened (not from the widget), the content is not obscured, but has a different issue in that there is too much space between the app bar and the start of the content:

Screenshot_20170515-125737.png (1×1 px, 245 KB)

Occurring on

Originally reported via OTRS, bug replicated on
Device = Pixel (Android 7.1.2)
Version = Wikipedia 2.5.195-alpha-2017-05-11

Details

Event Timeline

Mholloway renamed this task from BUG: When opening Today's Main page from the widget, top of Main page is cut-off to BUG: Main page display issues.May 25 2017, 2:48 PM
Mholloway updated the task description. (Show Details)

Change 357596 had a related patch set uploaded (by Dbrant; owner: Dbrant):
[apps/android/wikipedia@master] Fix top padding when viewing Main Page.

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

Change 357596 merged by jenkins-bot:
[apps/android/wikipedia@master] Fix top padding when viewing Main Page.

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

Mholloway renamed this task from BUG: Main page display issues to Top margin issues on enwiki main page.Jun 7 2017, 3:13 PM
Mholloway assigned this task to Dbrant.

Tested on Samsung-SM-JI20A Galaxy Express 3 (Android 6.0.1) and Wikipedia app 2.6.198-alpha-2017-06-12. This seems fixed as I don't see any margin spacing issues in the screencaps below (compared to the previous examples above) but I will let the staff decide.

T165307 From Explorer page.png (800×480 px, 136 KB)
T165307 Article.png (800×480 px, 159 KB)

ABorbaWMF subscribed.

Tested on Nexus 4 with Android 5.1 and Pixel with Android 7.1 on 2.6.198-beta-2017-06-09

Looks fixed to me