From a88f09469d22a5142c452c6d0ead4c86d0550a46 Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 22 Apr 2014 14:32:45 -0700 Subject: [PATCH] Adjust reviewer transaction icon in Differential Summary: Ref T4866. I did a fancy version of this but it looks pretty bad/confusing so here's a simple version. Fancy-but-whack version: {F146847} Test Plan: This version is like that, but just always uses `fa-user`. Reviewers: chad Reviewed By: chad Subscribers: epriestley Maniphest Tasks: T4866 Differential Revision: https://secure.phabricator.com/D8840 --- .../differential/storage/DifferentialTransaction.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/applications/differential/storage/DifferentialTransaction.php b/src/applications/differential/storage/DifferentialTransaction.php index 36b4556fd4..a87072458c 100644 --- a/src/applications/differential/storage/DifferentialTransaction.php +++ b/src/applications/differential/storage/DifferentialTransaction.php @@ -370,6 +370,11 @@ final class DifferentialTransaction extends PhabricatorApplicationTransaction { case DifferentialAction::ACTION_CLAIM: return 'fa-flag'; } + case PhabricatorTransactions::TYPE_EDGE: + switch ($this->getMetadataValue('edge:type')) { + case PhabricatorEdgeConfig::TYPE_DREV_HAS_REVIEWER: + return 'fa-user'; + } } return parent::getIcon();