Item1951: "Debug" topic diff should treat topic text as preformatted

pencil
Priority: Normal
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Engine
Component: rdiff
Branches:
Reported By: MichaelTempest
Waiting For:
Last Change By: KennethLavrsen
I had to "raw topic view" the two revisions I was interested in and compare them by eye to see certain changes, because the "debug" topic diff view hides some changes. I think this is because the topic text is shown in div blocks, instead of in pre blocks. (Perhaps there is a css way to do this - in which case please enlighten me smile )

I am not entirely sure which component produces the "debug" diff, but the problematic URL is http://my.server/foswiki/bin/rdiff/Web/Topic

As far as I can tell, rdiff is the "core" diff functionality. Since the CompareRevisionsAddOn is a core plugin in 1.1, and it provides a friendlier user-level diff, I think the rdiff output should become more useful for debugging.

-- MichaelTempest - 21 Aug 2009

I can achieve what I want by changing pub/System/PatternSkin/style.css, line 53, from this:
pre, code, tt {
    font-family: "Bitstream Vera Sans Mono","Andale Mono",Courier,monospace;
    font-size:86%;
}
to this:
pre, code, tt, .foswikiDiffDebug {
    font-family: "Bitstream Vera Sans Mono","Andale Mono",Courier,monospace;
    white-space:pre;
    font-size:86%;
}

I could also put a rule around line 1123, where the rest of the CSS for the rdiff template is. Which is better practice?

-- MichaelTempest - 23 Aug 2009

I have removed white-space:pre from pre, code, tt because this made code text non-wrapping.

-- ArthurClemens - 13 Sep 2009

ItemTemplate edit

Summary "Debug" topic diff should treat topic text as preformatted
ReportedBy MichaelTempest
Codebase 1.0.6, trunk
SVN Range Foswiki-1.0.0, Thu, 08 Jan 2009, build 1878
AppliesTo Engine
Component rdiff
Priority Normal
CurrentState Closed
WaitingFor
Checkins distro:b04f2a8ba407 distro:9d4bb711ea7b distro:aa18a74b9d14 distro:30299859fe92
TargetRelease minor
ReleasedIn 1.1.0
Topic revision: r11 - 04 Oct 2010, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy