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.
--
Ilmari Karonen