From 5b6b2ee8505f47cdba23b7ceaa5cf754a65f547e Mon Sep 17 00:00:00 2001 From: epriestley Date: Tue, 27 Jan 2015 14:49:42 -0800 Subject: [PATCH] Give Maniphest a title glyph Summary: Ref T7055. Apparently we just never had one? I feel like I'm crazy. But I can't find any trace in the logs. I'm actually not 100% sold on this being better because it's a color glyph on OSX and those feel a little out of place / tacky to me compared to the black-and-white ones. So I'd be fine with just leaving it off, too. Clearly not important if no one noticed it until I caught it in T7055. Test Plan: {F276917} Reviewers: chad Reviewed By: chad Subscribers: epriestley Maniphest Tasks: T7055 Differential Revision: https://secure.phabricator.com/D11524 --- .../maniphest/application/PhabricatorManiphestApplication.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/applications/maniphest/application/PhabricatorManiphestApplication.php b/src/applications/maniphest/application/PhabricatorManiphestApplication.php index 7588d26309..1df47ad390 100644 --- a/src/applications/maniphest/application/PhabricatorManiphestApplication.php +++ b/src/applications/maniphest/application/PhabricatorManiphestApplication.php @@ -22,6 +22,10 @@ final class PhabricatorManiphestApplication extends PhabricatorApplication { return 'fa-anchor'; } + public function getTitleGlyph() { + return "\xE2\x9A\x93"; + } + public function isPinnedByDefault(PhabricatorUser $viewer) { return true; }