Page MenuHomePhabricator
Differential D10948

Update message and notification icons to use fonts
ClosedPublic
Actions

Authored by chad on Dec 8 2014, 9:50 PM.
Tags
None
Referenced Files
F13268025: D10948.diff
Wed, May 29, 4:31 AM
F13253512: D10948.diff
Sat, May 25, 2:43 AM
F13241794: D10948.id.diff
Wed, May 22, 11:30 PM
F13232124: D10948.diff
Tue, May 21, 1:02 AM
F13229266: D10948.id26293.diff
Mon, May 20, 3:19 PM
F13213325: D10948.diff
Fri, May 17, 7:38 AM
Unknown Object (File)
Tue, May 7, 6:10 AM
Unknown Object (File)
Fri, May 3, 3:48 AM
View All Files
Subscribers
epriestley
Korvin
richardvanvelzen

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP34fb98da66ea: Update message and notification icons to use fonts
Summary

Cleans up spacing, updates to fonts instead of images. Fixed some mobile issues.

Test Plan

Test with and without counts on desktop, tablet, mobile. Test layout in FF, Chrome, IE.

Screen_Shot_2014-12-08_at_1.48.57_PM.png (178×528 px, 46 KB)

Screen_Shot_2014-12-08_at_1.49.08_PM.png (162×528 px, 40 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 26292.Dec 8 2014, 9:50 PM
chad retitled this revision from to Update message and notification icons to use fonts.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptDec 8 2014, 9:50 PM
chad edited the test plan for this revision. (Show Details)Dec 8 2014, 9:51 PM
epriestley accepted this revision.Dec 8 2014, 9:51 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Dec 8 2014, 9:51 PM
chad added a comment.EditedDec 8 2014, 9:52 PM

Unfortunately there isn't a usable in the font set, so changed to .

Closed by commit rP34fb98da66ea: Update message and notification icons to use fonts (authored by chad, committed by chad). · Explain WhyDec 8 2014, 9:53 PM
This revision was automatically updated to reflect the committed changes.
richardvanvelzen added a subscriber: richardvanvelzen.Dec 8 2014, 10:27 PM

I will miss the flame

Revision Contents
Changeset List

Diff 26293

View Options

resources/celerity/map.php

Loading...
View Options

resources/sprite/manifest/menu.json

Loading...
View Options

resources/sprite/menu_1x/conf-hover.png

Loading...
View Options

resources/sprite/menu_1x/conf-off.png

Loading...
View Options

resources/sprite/menu_1x/conf-unseen.png

Loading...
View Options

resources/sprite/menu_1x/seen_have_unread.png

Loading...
View Options

resources/sprite/menu_1x/seen_read_all.png

Loading...
View Options

resources/sprite/menu_1x/unseen_any.png

Loading...
View Options

resources/sprite/menu_2x/conf-hover.png

Loading...
View Options

resources/sprite/menu_2x/conf-off.png

Loading...
View Options

resources/sprite/menu_2x/conf-unseen.png

Loading...
View Options

resources/sprite/menu_2x/seen_have_unread.png

Loading...
View Options

resources/sprite/menu_2x/seen_read_all.png

Loading...
View Options

resources/sprite/menu_2x/unseen_any.png

Loading...
View Options

src/applications/celerity/CeleritySpriteGenerator.php

Loading...
View Options

src/view/page/menu/PhabricatorMainMenuView.php

Loading...
View Options

webroot/rsrc/css/application/base/main-menu-view.css

Loading...
View Options

webroot/rsrc/css/sprite-menu.css

Loading...
View Options

webroot/rsrc/image/sprite-menu-X2.png

Loading...
View Options

webroot/rsrc/image/sprite-menu.png

Loading...
Phabricator · Built by Phacility