Page MenuHomePhabricator

Revert changes to Diffusion blame view
ClosedPublic

Authored by epriestley on Oct 31 2017, 6:04 PM.
Tags
None
Referenced Files
F19931878: D18746.id44999.diff
Fri, Apr 10, 5:31 AM
F19931363: D18746.diff
Thu, Apr 9, 7:00 PM
F19884080: D18746.id44999.diff
Sat, Mar 21, 12:45 AM
F19828931: D18746.id44999.diff
Mar 9 2026, 11:53 PM
F19804225: D18746.id44995.diff
Mar 2 2026, 6:08 AM
F19802148: D18746.id44999.diff
Mar 1 2026, 1:46 PM
F19799281: D18746.id44999.diff
Feb 27 2026, 11:14 PM
F19103511: D18746.diff
Dec 5 2025, 10:51 AM
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