Page MenuHomePhabricator

Allow DocumentView to render with a curtain, and make Phriction use a curtain
ClosedPublic

Authored by epriestley on Aug 28 2018, 9:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 28, 4:18 AM
Unknown Object (File)
Wed, Nov 27, 10:50 PM
Unknown Object (File)
Fri, Nov 22, 9:13 AM
Unknown Object (File)
Thu, Nov 21, 9:29 AM
Unknown Object (File)
Thu, Nov 14, 5:55 PM
Unknown Object (File)
Thu, Nov 14, 4:53 AM
Unknown Object (File)
Thu, Nov 14, 12:01 AM
Unknown Object (File)
Oct 1 2024, 3:20 PM
Subscribers
None

Details

Summary

Depends on D19616. Ref T13077. Fixes T8172. In the last round of design updates, a lot of actions got stuffed into "Actions" menus.

I never really got used to these and think they're a net usability loss, and broadly agree with the feedback in T8172. I'd generally like to move back toward a state where actions are available on the page, not hidden in a menu.

For now, just put a curtain view on these pages. This could be refined later (e.g., stick this menu to the right hand side of the screen) depending on where other Phriction changes go.

(Broadly, I'm also not satisfied with where we ended up on the fixed-width pages like Diffusion > Manage, Config, and Instances. In contrast, I do like where we ended up with Phortune in terms of overall design. I anticipate revisiting some of this stuff eventually.)

Test Plan
  • Looked at Phriction pages on desktop/tablet/mobile/printable -- actions are now available on the page.
  • Looked at other DocumentView pages (like Phame blogs) -- no changes for now.

Diff Detail

Repository
rP Phabricator
Branch
phriction2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20699
Build 28135: Run Core Tests
Build 28134: arc lint + arc unit

Event Timeline

Updating unit results 504'd for some reason. I'll keep an eye on it but...

iiam

This revision was not accepted when it landed; it landed in state Needs Review.Aug 28 2018, 9:58 PM
This revision was automatically updated to reflect the committed changes.