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
This commit is contained in:
@@ -22,6 +22,10 @@ final class PhabricatorManiphestApplication extends PhabricatorApplication {
|
|||||||
return 'fa-anchor';
|
return 'fa-anchor';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function getTitleGlyph() {
|
||||||
|
return "\xE2\x9A\x93";
|
||||||
|
}
|
||||||
|
|
||||||
public function isPinnedByDefault(PhabricatorUser $viewer) {
|
public function isPinnedByDefault(PhabricatorUser $viewer) {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user