This UI approach seems to have a number of issues.

• It doesn't scale well (lots of long comments are going to drastically increase page height, perception of where user is in the review process)
• Differential also is a 2-up display, with code selection spanning multiple-lines sometimes
• We have an expectation of well thought out, even marked up text
• Providing a more 'Table of Contents' view is likely more efficient over repeating every code blocks multiple times

On the other hand
I do like the idea of having 'Dialogs' possibly available as a button/icon next to the line numbers. This gives the user the choice to see the entire context (link to the previous diff) or just a snippet (pop a dialog). Would have to get @epriestley's thoughts.

I think we'll probably revisit inlines, but likely not until after T1460 (mostly a product blocker) and T2009 (mostly a technical blocker). Particularly, T1460 will create some new UI that forces us to take another look at some of this stuff.

Also -- if the comment is for a block of code that is not changed between two diffs, probably make sense to also show it in the latest diff?

This isn't possible in the general case (diffs A and B may be against completely different code), and I worry that doing it conditionally will be confusing. Users already find inlines relatively confusing (T3669, maybe T3639), and having them sometimes port forward and sometimes vanish threatens to make this more complicated. I don't think this is impossible, but I do think it's challenging to implement while making it clear to the user when comments will port forward, why they don't when they don't, and where the went.

Thanks @epriestley! I love the idea in T1460 a lot. Also what @chad suggested makes a lot of sense -- make a popup window with some context would be really awesome.

◀ Merged tasks: T5743.

I'm not particularly sold on putting context in the timeline, particularly because I think we've received about as much feedback about Differential having "too much stuff" as we have about this context being desirable.

However, I think this information is probably useful to add:

• Is the inline marked done?
• Checkbox to let you mark the inline done?
• Link to the inline in whatever view comes out of T8250.
• Marker if the inline is a reply?

I'm just going to merge this into T8250. I'm still unsure what's to become of this, but I think however that resolves will cover this.