Refactored from
Item1783
The problem
My users are so sad that the
CompareRevisionsAddOn? no longer works after upgrading to 4.0.2. It gives a diff display which is very superior to the rdiff display. Since 4.0.2 we just get a blank screen when trying to use it.
The solution provided by
TWiki:Main.SteffenPoulsen
and tested by
TWiki:Main.KennethLavrsen
From 1783
QUOTE
Took a quick shot at
CompareRevisionsAddOn, attached updated versions of interweave and compare.
Never used this plugin before, so I don't know if it displays as expected now, but it displays something. Perhaps you will SVN commit / re-package / talk to author if it works OK,
KJL?
Don't know about the view_cra, I believe the change should be applied to viewtopicactionbuttons.pattern.tmpl now, if wanted. If each of the single "<" should go to compare instead of rdiff, of course it's in lib/TWiki/UI/View.pm the rdiff should be changed to compare - though that doesn't seem to be suggested in the plugin topic anywhere).
--
SP
UNQUOTE
Hey. It works. Wow. Thanks. You just earned yet another beer, Steffen.
I will checkin the changes and contact author.
--
KJL
Nice combo, this. Updated
CompareRevisionsAddOn and
HistoryPlugin to be 4.0.2 compatible, take them for a spin.
TODO: Add screenshots to SVN, add wikipedia-like javascript to
HistoryPlugin.
SVN 9759.
--
SP
So should
viewtopicactionbuttons_cra
be merged with
viewtopicactionbuttons
?
AC
It hope it already is, but would appreciate a check on it. I will remove the file from the plugin at 4.0.3 or new PatternSkin release (could also be removed and replaced with some doc explaining to get it from SVN, but for now it's a local copy in the .zip).
Btw: Saw you noticed the
very black table
in the IRC log last night, would be happy if something could be done about that, I've given up on that [hint, hint]
--
SP
- No,
viewtopicactionbuttons_cra
is not yet merged.
- Huh! There is template-specific stuff in
Compare.pm
(my $revisions). Not suitable for other skins, and it may break easily. It is even out of tune with pattern skin already.
- I would not put the 2 compare links in between their own separators, like
| History | Side-by-side |
, but: | History: Mixed, Side by side |
- Why is the newest version at the left, not at the right as rdiff does? The right seems more logical to me. See also the Wikipedia compare
with the newest version at the right.
- When on the side-by-side comparison page, why is there a link "History" that points to
rdiff
? Why not keep the links "Mixed" and "Side-by-side" with the current 'view' in selected state (not a link)?
- This addon should ship with
CompareRevisionsAddOnDemoTopic.txt,v
to actually see the differences.
- Shouldn't pattern skin's More page be updated as well with CompareRevisions options (
compare
instead of rdiff
)
- I will look at the table. It just has a very black border that is out of tune. It appears that the borders are defined by table cells. I need a css class around the table! So I can define in my stylesheet
.compareRevisionsTable table
.
- What's a wikipedia-like javascript?
AC
I think it would be even better (simpler) if the History link would stay as it is (for the user) while only the link changes to a "Compare" page with 2 tabs: "Mixed" and "Side by side". The compare page opens the tab that the user viewed last (and is read from a cookie).
That way we keep the bottom links simple (without users having to know what is "Mixed" or "Side by side". And the interface stays consistent, which is nice for skin builders.
AC
Is this still live? there seems to have been a lot of checkins, but no feedback. Assigning to Kenneth for closure.
CC
Closing
KJL