Page MenuHomePhabricator

Add an explicit "max-width" to PHUIDocumentPro pages to force large tables to scroll
ClosedPublic

Authored by epriestley on Oct 1 2018, 6:08 PM.
Tags
None
Referenced Files
F15438718: D19723.id47122.diff
Wed, Mar 26, 3:01 AM
F15438194: D19723.id47119.diff
Tue, Mar 25, 11:42 PM
F15435877: D19723.id47119.diff
Tue, Mar 25, 10:39 AM
F15433005: D19723.id.diff
Mon, Mar 24, 8:50 PM
F15425820: D19723.diff
Sun, Mar 23, 6:18 AM
F15385235: D19723.id.diff
Fri, Mar 14, 10:02 PM
Unknown Object (File)
Mar 3 2025, 7:58 AM
Unknown Object (File)
Mar 1 2025, 10:13 PM
Subscribers
None

Details

Summary

Ref T13202. See https://discourse.phabricator-community.org/t/phriction-page-controls-lost-after-creating-very-wide-table/1961.

If you put a very wide table in the markup for a new-layout Phriction page, it can push the actions element off screen to the right.

Tables already get a scrollbar if encouraged strongly enough; add a max-width to encourage them.

Test Plan
  • Viewed pages with a large wrappable and non-wrappable content on mobile, tablet, and desktop.

Screen Shot 2018-10-01 at 11.06.29 AM.png (1×1 px, 144 KB)

Diff Detail

Repository
rP Phabricator
Branch
wiki1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 20929
Build 28449: Run Core Tests
Build 28448: arc lint + arc unit