I've downloaded the lastest version of Diff and have a question.

1) When I look at revisions, it gives me an HTML view. Is there any way for the HTML to not show as code?

Thanks,
MatthewS

Comments

moshe weitzman’s picture

no. we show the differences in user input between two revisions. if users entered HTML, thats whats shown. The old way didn't really look or work very good.

toemaz’s picture

I agree on Moshe's feedback.

matthews’s picture

Aha. Ok. I'm using TinyMCE to make text look "good" to clients (both internal to my organization and external to) so it seems that we might need to choose to not use TinyMCE if we want to use Diff in a readable way.

rötzi’s picture

fool2’s picture

A possible solution:

Convert formatting to some readable, non-html entity.

For instance,

<font color="blue">

could be converted to

[COLOR:BLUE]

which, when diffed, (say if you changed the color to red) would show

[COLOR:<html formatting for diff minus>BLUE</><html formatting for diff plus>RED</>]

I think this would still alow people to track formatting changes without freaking the layperson out with html code (especially broken html code).

We shouldn't really be SEEING the formatting (aside from linebreaks) anyway as there is always the possibility that diff formatting conflicts with the formatting in the document.

The possible issues, however:

-Closing tags might still freak people out (how to encapsulate a formatted piece of text without them?)
-It would still pale in comparison to tools like "track changes"
-It might be more work than its worth

jonnyz’s picture

rotzi did a great job at addressing this issue using a setting in the admin area that allows you to specify whether you want the user to see all the markup or strip out html. I tested it out at some point and it worked great. He mentioned that he would work on getting it integrated at the end of march but i don't know where it stands. it had a bunch of other great changes as well built into that release. moshe...maybe you could work with rotzi to integrate? i don't code myself but would be more than happy to help test.

jz

moshe weitzman’s picture

Status: Active » Closed (fixed)

4.7 is old, and i'm not sure there is any todo here. i think the current diffs work nicely. please reopen as needed.

yurtboy’s picture

is this patch still working for the latest release of this module and drupal 5.x?