Page MenuHomePhabricator

Differential Changeset header icons
ClosedPublic

Authored by chad on Oct 15 2013, 6:02 AM.
Tags
None
Referenced Files
F14038206: D7320.id16492.diff
Sun, Nov 10, 10:08 PM
F14023523: D7320.id16488.diff
Thu, Nov 7, 2:34 AM
F13981986: D7320.id16488.diff
Sat, Oct 19, 8:30 PM
F13981823: D7320.id16487.diff
Sat, Oct 19, 7:19 PM
F13981822: D7320.diff
Sat, Oct 19, 7:19 PM
F13981812: D7320.id.diff
Sat, Oct 19, 7:16 PM
F13981797: D7320.id16487.diff
Sat, Oct 19, 7:08 PM
F13981794: D7320.id16488.diff
Sat, Oct 19, 7:06 PM

Details

Reviewers
epriestley
btrahan
Commits
Restricted Diffusion Commit
rP451dfb9afbb0: Differential Changeset header icons
Summary

Adds filetype icons, applying to differential file headers. The main issue is with all the lightening, I wanted something to still anchor 'new file' on the page and adding a sharp icons does that pretty well for me. Feedback is cool too.

Test Plan

Add some new icons, test in previous commits.

Diff Detail

Branch
diff-headers
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

chad updated this revision to Unknown Object (????).Oct 15 2013, 6:07 AM
  • Missed update

Oh man I love icons.

One thought is that the file tree and Diffusion currently have different icons and maybe we should merge them at some point, but these look good to me.