@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
Use People Query's "Disabled" style also in Project Members list
Summary:
Add more consistency where to look for the Disabled status of a listed user and how that indicator looks like (icon use).
Closes T16193
Test Plan: Go to the Members of a Project after disabling one of those member user accounts.
Reviewers: O1 Blessed Committers, valerio.bozzolan
Reviewed By: O1 Blessed Committers, valerio.bozzolan
Subscribers: tobiaswiese, valerio.bozzolan, Matthew, Cigaryno
Maniphest Tasks: T16193
Differential Revision: https://we.phorge.it/D26219
+7
-7
+7
-7
src/applications/project/view/PhabricatorProjectUserListView.php
+7
-7
src/applications/project/view/PhabricatorProjectUserListView.php
···
115
115
116
116
$item
117
117
->setDisabled(true)
118
-
->addAttribute(pht('Disabled'));
119
-
} else {
120
-
$icon = id(new PHUIIconView())
121
-
->setIcon($handle->getIcon());
118
+
->addIcon('fa-ban', pht('Disabled'));
119
+
}
122
120
123
-
$subtitle = $handle->getSubtitle();
121
+
$icon = id(new PHUIIconView())
122
+
->setIcon($handle->getIcon());
123
+
124
+
$subtitle = $handle->getSubtitle();
124
125
125
-
$item->addAttribute(array($icon, ' ', $subtitle));
126
-
}
126
+
$item->addAttribute(array($icon, ' ', $subtitle));
127
127
128
128
if ($supports_edit && !$is_panel) {
129
129
$remove_uri = $this->getRemoveURI($user_phid);