Page MenuHomePhabricator

Differential Changeset header icons
ClosedPublic

Authored by chad on Oct 15 2013, 6:02 AM.
Tags
None
Referenced Files
F15511089: D7320.diff
Wed, Apr 16, 11:54 PM
F15461486: D7320.id16488.diff
Tue, Apr 1, 8:39 AM
F15423749: D7320.id16488.diff
Mar 22 2025, 5:35 PM
F15422190: D7320.id16492.diff
Mar 22 2025, 6:27 AM
F15420637: D7320.id16487.diff
Mar 21 2025, 3:41 PM
F15405017: D7320.id.diff
Mar 18 2025, 10:01 AM
F15380004: D7320.id16492.diff
Mar 14 2025, 12:47 AM
Unknown Object (File)
Mar 2 2025, 1:57 AM

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.