Page MenuHomePhabricator

Differential Changeset header icons
ClosedPublic

Authored by chad on Oct 15 2013, 6:02 AM.
Tags
None
Referenced Files
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
F13981720: D7320.diff
Sat, Oct 19, 6:38 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

Lint
Lint Skipped
Unit
Tests Skipped

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.