coauthor icon indicating copy to clipboard operation
coauthor copied to clipboard

diffs in notifications

Open edemaine opened this issue 7 years ago • 2 comments

Email notifications should show diffs instead of full messages (when appropriate). Related to #11.

  • [ ] title should be character-level diff'd
  • [ ] body should be diff'd (after or before formatting? maybe after in HTML, before in ASCII?)
  • [x] tags should be unordered-list diff'd (cross out old, highlight new)
  • [ ] format should have crossed-out old value and highlighted new value

~~The main challenge here is figuring out what the message was before the given sequence of diffs. #34 might be useful for this.~~

One approach would be to look for unchanged paragraphs, list items, and table rows, and convert them to "..."s. <(p|tr|li)>(.*)</\1> but balanced.

edemaine avatar Nov 24 '16 18:11 edemaine

I've looked around for a good HTML diffing package that can make a visual indication of the changes. Unfortunately, none seem to have tested out to be useful. I think we simply would like to look at the sequence of text nodes in the DOM, and do greedy diff -- either treating the text nodes as lines, or maybe working on the concatenation of the text nodes?

edemaine avatar Aug 01 '17 03:08 edemaine

Hi there! First of, thanks for putting Coauthor together and release it to the public :-) Any news on seeing diff between history versions? Or displaying the diff in the UI ( #11 )? Either of these would be quite useful imo: when thread get big, the history navigation is not really practical.

romain-jacob avatar May 27 '20 09:05 romain-jacob