Page MenuHomePhabricator

Revert changes to Diffusion blame view
ClosedPublic

Authored by epriestley on Oct 31 2017, 6:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 11, 2:34 AM
Unknown Object (File)
Apr 10 2026, 5:31 AM
Unknown Object (File)
Apr 9 2026, 7:00 PM
Unknown Object (File)
Mar 21 2026, 12:45 AM
Unknown Object (File)
Mar 9 2026, 11:53 PM
Unknown Object (File)
Mar 2 2026, 6:08 AM
Unknown Object (File)
Mar 1 2026, 1:46 PM
Unknown Object (File)
Feb 27 2026, 11:14 PM
Subscribers
None

Details

Summary

Ref PHI174. This reverts most of these changes:

These changes made the Diffusion blame view very similar to GitHub's blame view. See D18452 for a before/after of the bulk of these changes; the other revisions are bugfixes.

I think this was generally a step backward, and not motivated by solving a specific problem. I've found the new UI less usable than the old one, and at least one install (see PHI174) also has.

In particular, the revision/commit titles are very bulky and not terribly useful; the date column also isn't terribly useful; the "age" color actually IS pretty useful and was heavily de-emphasized.

I've kept one bugfix here (missing 'a' tag type) and kept the upgraded icon for "Skip Past This Commit".

I'm going to follow this up with some additional changes:

  • Show a small author profile icon, similar to GitHub, to address PHI174 more directly.
  • Try a zebra-stripe on blocks of rows to make it more clear where changes affected by a particular commit begin and end.
  • Try a hue shift, not just a brightness/saturation shift, to make the "age" color more distinct.
  • Try computing colors as even steps, not based purely on age. Currently, if a file has one long-distant commit and several recent commits, all the recent ones show up as very bright green. I think this would probably be more useful if they were distributed more evenly across the available color bands.
Test Plan

Viewed blame views in Diffusion, saw a more compact UI similar to the old UI.

Screen Shot 2017-10-31 at 11.03.19 AM.png (266×766 px, 67 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable