Page MenuHomePhorge

Fix homepage in mobile
Closed, ResolvedPublic

Description

The home of Phabricator/Phorge is just broken on mobile.

As far as I can remember, it just shows the menu:

Phorge home mobile.png (537ร—384 px, 29 KB)

But it's really duplicated:

Phorge home mobile open menu.png (537ร—384 px, 47 KB)

So, you never see the contents. You just see the menu.

What should happen instead

On mobile, the menu should be hidden, not the contents. So, you see the contents:

Phorge mobile After.png (537ร—384 px, 39 KB)

And, if you want, you can see the menu of course, as usual:

Phorge home After with menu.png (537ร—384 px, 50 KB)


I set this as high priority, since this is a really boring issue, even if it's a long-term issue.

Historical notes

Part of the regression was probably introduced here:

https://secure.phabricator.com/rPad01e26af71e7d7936dd2347eca47674255a7838
https://secure.phabricator.com/D17259
https://secure.phabricator.com/T11957

Problematic CSS:

webroot/rsrc/css/phui/phui-basic-nav-view.css
.phabricator-home.device-phone .phabricator-nav-content {
  display: none;
}

A/B Difference

Before D25107After D25107
Gitpull Before D25107.png (537ร—324 px, 30 KB)
Gitpull After D25107.png (537ร—324 px, 37 KB)

Event Timeline

valerio.bozzolan created this task.
valerio.bozzolan created this object in space S1 Public.
valerio.bozzolan lowered the priority of this task from High to Normal.

Yep but technically this is not resolved in the master branch. I see that Tasks are marked as resolved when their modification is landed in the master branch.

Also it's really nice when the change is landed in the master branch, and then Diffusion automatically closes the related Task (this one). I don't know if you are aware of this automation but it's really nice. Maybe we can keep open, just to see it in action again, when the change will be landed in master.