Page MenuHomePhabricator

Linking to lines in a diff
Closed, WontfixPublic

Description

Would be nice to have a feature similar to Github's link to line (https://github.com/gitlabhq/gitlabhq/pull/1937). epriestley thinks this is best done after T2009

Event Timeline

dougluce raised the priority of this task from to Needs Triage.
dougluce updated the task description. (Show Details)
dougluce added a project: Differential.
dougluce added a subscriber: dougluce.

Note that you can already link to a line in the Diffusion browse view, where this problem is much simpler. Here's an example:

https://secure.phabricator.com/diffusion/P/browse/master/src/applications/pholio/controller/PholioMockViewController.php;3e946140d36fe83c321ffd830512a85cbf61d77b$8-10

This is slightly more difficult in the diff views and in Differential, primarily because we need to expand context when a user links to a line which is hidden behind "show more context".

epriestley claimed this task.

(Internally, see PHI2020.)

Currently, I haven't seen requests for this that clearly outline a compelling use case.

I worry that the use case I have seen ("send links to someone else in Slack") may actually be a bad use case, and authors receiving those links might often prefer to receive inline comments (asynchronously) instead of a synchronous chat notification. That is, I generally expect engineers and organizations value the asynchronous nature of Differential code review and want to avoid interrupting engineers when not necessary, and that features which convert asynchronous workflows into synchronous workflows should be presumed to be undesirable in the absence of robust use cases.

In an extreme case, we can imagine a "Start Zoom Call" button on Differential, which called the author and all the reviewers into a synchronous video chat. This would probably be a disastrously bad feature that wasted a terrible amount of time on the balance, even if some users really liked it.

If your organization would benefit from this feature (linking to lines in diff), please raise use cases via your support channel. It would be particularly helpful to understand why the value of this feature outweighs the implied costs of converting asynchronous workflows into synchronous workflows (or why your organization doesn't believe interrupting engineers is costly, or whatever other context impacts this workflow). If possible, I'd prefer to address whatever use cases exist in other ways -- for example, by making it easier to link to an inline comment, rather than a particular line.

Another thought is that some of the use case might be social -- it might feel a little embarrassing to ask (or point out) a silly question/mistake? I don't have any evidence of this, but if this is part of the use case, I'd like to at least consider something like a "private inline" between the reviewer and the author. I don't like the idea of this feature very much, but I like interrupting authors with Slack notifications even less.

(A robust use case may also evolve from the current discussion in PHI2020 and I may pursue this change once that develops; if so, I'll file some more-tailored upstream followup variant of this.)