Page MenuHomePhabricator

Put application curtain panels above extension curtain panels
ClosedPublic

Authored by epriestley on Fri, May 1, 2:26 PM.

Details

Summary

Ref T13528. The original rationale here was that it's easier to find items at the bottom of the curtain than somewhere in the middle, since they're in a more clearly predictable visual location.

This might be true in some sense, but user feedback about this has fairly consistently indicated that the layout is surprising. Try the other order. See also D20967 for some discussion.

In practice, this primarily moves "Author / Assigned" above other panel elements in Maniphest.

Test Plan

Looked at tasks, aw "Author / Assigned" above "Tags / Subscribers".

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Fri, May 1, 2:26 PM
epriestley requested review of this revision.Fri, May 1, 2:27 PM
This revision was not accepted when it landed; it landed in state Needs Review.Fri, May 1, 4:11 PM
This revision was automatically updated to reflect the committed changes.