Steve Bennett wrote:
Hi again, Ok I had my own attempt at adding "group edits". It works, but has the obvious limitation that displaying the diff across a group of collapsed diffs shows not quite what you're expecting. In other words:
Time t0 User A comment1 Time t1 User A Time t2 User A comment2 Time t3 User B comment 3
It shows this as Time t0 User A comment1//---//comment2 Time t3 User B
So if you attempt to compare between t0 and t3, you of course get all the changes made in t1, t2 and t3, whereas you probably just wanted t3. Not sure how to deal with that one.
It's probably better to give diffs against the *last* edit in each group. Doing so not only solves this particular case, but is probably more meaningful in general.
I haven't taken a look at the code yet; if I get around to it, I may post a patch for this.