Page MenuHomePhabricator

Inline comments should be threaded in the Diffusion top level view.
Closed, WontfixPublic

Description

When there is a discussion happening in the code review about a specific section of the code, all inline comments that apply to the same lines should be grouped together so that it is possible to see the conversation as it develops.
Currently you can only see this by navigating to the actual lines in the review, but the comments are no longer visible after the code is updated. It seems like it would be easier to follow the flow of the review if all comments that were related were grouped together rather than being in strict chronological order.

Event Timeline

epriestley closed this task as Wontfix.Jan 6 2015, 1:13 AM
epriestley claimed this task.
epriestley added a subscriber: epriestley.

Some discussion in T1460, but we don't plan to pursue this, at least as the primary view.

Notably, threading comments makes it very difficult to identify which comments are new.

I agree about that "new" thing.
Do you guys have plans to persist threaded view in code itself through rev updates?