Page MenuHomePhabricator

Restore the "buoyant" header in Differential
ClosedPublic

Authored by epriestley on May 17 2017, 9:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 22, 10:42 PM
Unknown Object (File)
Thu, Dec 19, 7:47 AM
Unknown Object (File)
Tue, Dec 17, 7:43 AM
Unknown Object (File)
Tue, Dec 17, 7:09 AM
Unknown Object (File)
Sat, Dec 14, 4:38 PM
Unknown Object (File)
Wed, Dec 11, 5:36 PM
Unknown Object (File)
Wed, Dec 11, 7:12 AM
Unknown Object (File)
Wed, Dec 11, 2:52 AM
Subscribers
None
Tokens
"Love" token, awarded by avivey."Dat Boi" token, awarded by ftdysa.

Details

Summary

Fixes T1591. This was removed long ago because it was a mess to implement and caused a bunch of weird issues, and also my tolerance for dealing with weird JS issues was much, much lower.

I have now survived the fires of JX.Scrollbar and would love to address 200 small nitpicks about obscure browser behaviors on Linux, so open the floodgates again.

A secondary goal here is to create room to add a global view state menu on the right, with 300 options like "hide all inlines", "hide done inlines", "hide collapsed inlines", "hide ghosts", "show ghosts", "enable filetree", "disable filetree", etc, etc. Not sure how much of this I'll actually do. I have one more experiment I want to try first.

Test Plan

Screen Shot 2017-05-17 at 2.38.36 PM.png (760×1 px, 114 KB)

Diff Detail

Repository
rP Phabricator
Branch
inline25
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17076
Build 22835: Run Core Tests
Build 22834: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.May 18 2017, 3:47 PM

I expect to keep tweaking this and receive 20 pieces of conflicting feedback about it, but I'm more confident we can make it stick now than I was in 2012.

This revision was automatically updated to reflect the committed changes.

I like this a lot and think we should expand it with more features (like, how many inlines are complete, etc).

tempted to make it inversely colored too

so many design possibilities