From e2fcc3c1879c280b963a30f4fe28cc6db9306062 Mon Sep 17 00:00:00 2001 From: Chad Little Date: Thu, 19 Feb 2015 07:03:18 -0800 Subject: [PATCH] Touch up Audit/Commit List UI Summary: Fixes a few issues. The author of the commit is more prominent / not cut off. Auditors is in a more consistent location. More space is available for reasons. Commits by themselves look much less janky. Only downside is actual Audits are now 3 lines vs. 2, but the extra space is used well. Test Plan: Test list of audits and commits. {F309237} Reviewers: btrahan, epriestley Reviewed By: epriestley Subscribers: Korvin, epriestley Differential Revision: https://secure.phabricator.com/D11817 --- src/applications/audit/view/PhabricatorAuditListView.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/applications/audit/view/PhabricatorAuditListView.php b/src/applications/audit/view/PhabricatorAuditListView.php index ac9dbf2322..6fc54f2e5f 100644 --- a/src/applications/audit/view/PhabricatorAuditListView.php +++ b/src/applications/audit/view/PhabricatorAuditListView.php @@ -152,10 +152,10 @@ final class PhabricatorAuditListView extends AphrontView { ->addAttribute($status_text) ->addAttribute($reasons) ->addIcon('none', $committed) - ->addByline($author_name); + ->setSubHead(pht('Author: %s', $author_name)); if (!empty($auditors)) { - $item->addAttribute(pht('Auditors: %s', $auditors)); + $item->addByLine(pht('Auditors: %s', $auditors)); } $list->addItem($item);