Homec4science

Remove "days fresh" / "days stale" indictator in Differential revision list

Authored by epriestley <git@epriestley.com> on May 16 2016, 18:43.

Description

Remove "days fresh" / "days stale" indictator in Differential revision list

Summary:
Ref T10939. I'm not totally opposed to the existence of this element, but I think it's the kind of thing that would never make it upstream today. I think this should just be a T418 custom sort of thing in the long run, not a mainline upstream feature.

Overall, I think this thing is nearly useless and just adds visual clutter. My dashboard is about 100% red. This also sort of teaches users that it's fine to let revisions sit for a couple of days, which isn't what I'd like the UI to teach. Finally, removing it helps the UI feel a little less cluttered after the visually busy changes in D15926.

Test Plan: Grepped for removed config. Viewed revision list.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10939

Differential Revision: https://secure.phabricator.com/D15927

Details

Committed
epriestley <git@epriestley.com>May 16 2016, 19:47
Pushed
aubortJan 31 2017, 17:16
Parents
rPHd46378df208f: Modernize "Responsible Users" tokenizer and add "exact(user)" token
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH8a98868bfbc4: Remove "days fresh" / "days stale" indictator in Differential revision list (authored by epriestley <git@epriestley.com>).May 16 2016, 19:47