Differential is a code review tool.
Fri, Jun 1
May 12 2018
May 9 2018
D19442 is marked as fixing this, although it just raises the limit from 80 to 100. This span measures the number of characters between the first changed character and the last changed character (not the total number of characters on the line) and we haven't seen tons of interest in this over the last several years so I think few installs are running into issues with it even at 80. Bumping it to 100 should give us a bit more breathing room but I don't currently plan to make it configurable (see T8227).
Apr 30 2018
Apr 28 2018
From PHI604, for completeness, on the newer behavior of "Hide Blame":
Apr 26 2018
Yeah, I'm looking at here, assuming it's ahead of last release.
Just to make sure we're talking about the same thing, that feedback is true even of the version at HEAD (blue/grey/orange) vs the version that originally shipped (green/grey/red)?
You could maybe leave the element off unless it is a larger change. I think my concern here is seeing it on every revision becomes something extra I have to mentally process, but if the intent is to either help show small changes, or large changes, it could just be visible in those edge cases. In those cases at least, the element provides context as to why it's shown at all vs. all the time?
I think there's possibly at least some argument that this element could serve noble goals:
I feel your instincts are correct, this information isn't useful in a list form and shouldn't be included. You could make the argument to include it in a hovercard, or bring it into the header on the diff page, and I think those are both fine. Outside of that this seems mostly geared towards avoiding code review, a practice I don't think is well aligned with the overall product goals.
Apr 24 2018
@epriestley, can you confirm that the above patch should disable the draft state? I have applied it to GHC's Phabricator deployment yet we are still seeing Differentials opened in draft state.
Apr 20 2018
Apr 19 2018
Apr 18 2018
Apr 17 2018
Apr 16 2018
See T13125 for a more detailed breakout of coverage plans.
Apr 15 2018
This particular page here is fataling:
Apr 12 2018
Yeah, this is a combination of a bulk editor bug (which lets you delete task titles) and legacy compatibility code for transaction title rendering (which treats null -> anything transactions as "created this thing.").
Oh, good, it's about time this task be created!