Ref T12136. This just yanks the band-aid off. Fundamentally these were useful well before Dashboards and advanced bucketing, but not so much any more. They also have some performance hit.
- Maniphest Tasks
- T12136: What to do with "Application Counts" with MenuItem
- rP20d1bb8fdf3d: Remove counts from home navigation
Add some tasks and diffs onto a new instance, see there is no count on the home menu bar.
I think we can get rid of a lot more stuff, like DifferentialApplication->loadNeedAttentionRevisions(), PhabricatorApplication->loadStatus() and callsites, the entire PhabricatorApplicationStatusView class, the stuff in People, Phrequent, Maniphest and Flags, probably some strings in USEnglishTranslation if you want to get ambitious, and the MAX_STATUS_ITEMS constant. Like seven million lines of code.
We could definitely integrate this into MenuItem (at least, if Home is staying wide? Maybe not if it's collapsing like Projects?) but ideally maybe no one notices or misses this stuff.
If we do ultimately put it back I want to try to put some proper caching behind it -- the cost is really disproportionate to the value, especially since 80% of the value is probably "is there a count or not" for users who find it valuable?
Maybe hold this for the release cut just in case? Although I wanted to try to get some more stuff in so I dunno what's going on there yet.
In pretty much all cases, I’d rather put time into fixing dashboards or
building Nuance, etc, if the count behind something is important.
Yes, I plan to hopefully have one wide menu for projects/home and roll them
Yeah, completely agreed.
Also I think I'm going to churn on some more stuff before cutting the release so just go ahead and kick this in, should have plenty of time to soak. I was just vaguely thinking I'd be cutting in like 15 minutes but I'm not going to try to rush it.
one wide menu for projects/home