Page MenuHomePhabricator

Give Maniphest a title glyph
ClosedPublic

Authored by epriestley on Jan 27 2015, 3:46 PM.
Tags
None
Referenced Files
F14059792: D11524.diff
Sun, Nov 17, 9:41 PM
F14050796: D11524.diff
Thu, Nov 14, 8:44 PM
F14033579: D11524.diff
Sat, Nov 9, 6:24 PM
F14021959: D11524.diff
Wed, Nov 6, 2:10 PM
F14009639: D11524.diff
Wed, Oct 30, 9:53 PM
F14002083: D11524.id.diff
Fri, Oct 25, 2:56 PM
F13966266: D11524.id27712.diff
Oct 16 2024, 7:17 AM
Unknown Object (File)
Oct 7 2024, 3:23 PM
Subscribers

Details

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

Screen_Shot_2015-01-27_at_7.45.41_AM.png (33×120 px, 6 KB)

Diff Detail

Repository
rP Phabricator
Branch
glyph1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4087
Build 4100: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Give Maniphest a title glyph.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 27 2015, 4:25 PM
This revision was automatically updated to reflect the committed changes.