Page MenuHomePhabricator

Misc Diffusion/Differential CSS tweaks
ClosedPublic

Authored by chad on Oct 15 2013, 6:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 31 2026, 7:44 AM
Unknown Object (File)
Mar 19 2026, 2:25 AM
Unknown Object (File)
Mar 5 2026, 9:33 PM
Unknown Object (File)
Mar 1 2026, 11:58 AM
Unknown Object (File)
Feb 6 2026, 7:02 AM
Unknown Object (File)
Jan 18 2026, 12:43 AM
Unknown Object (File)
Jan 10 2026, 10:49 PM
Unknown Object (File)
Nov 28 2025, 7:27 AM

Details

Summary

Various tweaks and fixes. Adds a File Contents view in Diffusion, normalizes spaces, colors.

Test Plan

tested differential and diffusion in my sandbox.

Diff Detail

Branch
diff-css
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

I'd like to make a more dramatic attack on that "giant mess of green" in the left columns at some point, but this is clearly a large step forward.

In particular, we currently show three columns:

| af832938 | D1234 | alincoln |

I think we could get away with this:

| D1234 / af832938 |

..and a hovercard/tooltip on the af832938 bit, showing the title/author/date/etc.

And then we could probably lose the shading on the number column, and make the "<<" a more prominent, easier-to-see/hit button.

yeah let me get that too. i was ignoring but it could use some love.

Oh, some of that might be a huge pain because of how it gets built -- if you run into a bunch of JS that makes zero sense, I can probably untangle that. This code got some cleanup recently but needs some more.

ok, i'll land and work on diffusion more. any other pain points?

That's the only one on my radar, I think.