@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Put application curtain panels above extension curtain panels
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".
Maniphest Tasks: T13528
Differential Revision: https://secure.phabricator.com/D21200
+2
-2
+2
-2
src/view/layout/PHUICurtainView.php
+2
-2
src/view/layout/PHUICurtainView.php
···
19
19
$panel = new PHUICurtainPanelView();
20
20
$this->addPanel($panel);
21
21
22
-
// By default, application panels go at the bottom of the curtain, below
22
+
// By default, application panels go at the top of the curtain, above
23
23
// extension panels.
24
-
$panel->setOrder(100000);
24
+
$panel->setOrder(1000);
25
25
26
26
return $panel;
27
27
}