Page MenuHomePhabricator

Scroll on overflow of property list (mobile)
ClosedPublic

Authored by chad on Sep 2 2015, 1:31 AM.

Details

Summary

Fixes T9314. Functionally phui-status-list should get moved off a table, but that's another day. This catches many other possible issues.

Test Plan

Review changes on a narrow browser.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7882
Build 8795: [Placeholder Plan] Wait for 30 Seconds
Build 8794: arc lint + arc unit

Event Timeline

chad retitled this revision from to Scroll on overflow of property list (mobile).
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

Not auto? This will always render a scrollbar, right? Even if the text fits? Should probably be auto?

This revision is now accepted and ready to land.Sep 3 2015, 2:10 PM

Oh, I didn't notice, but I built it on a trackpad so it probably did the auto hide stuff.

This revision was automatically updated to reflect the committed changes.