Page MenuHomePhabricator

Differential Changeset header icons
ClosedPublic

Authored by chad on Oct 15 2013, 6:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Feb 14, 2:38 PM
Unknown Object (File)
Fri, Feb 7, 9:42 AM
Unknown Object (File)
Fri, Feb 7, 9:42 AM
Unknown Object (File)
Fri, Feb 7, 9:42 AM
Unknown Object (File)
Fri, Feb 7, 8:37 AM
Unknown Object (File)
Fri, Feb 7, 8:01 AM
Unknown Object (File)
Jan 5 2025, 8:31 PM
Unknown Object (File)
Jan 4 2025, 1:26 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.