doc/todo/provide_inline_diffs_in_recentchanges.mdwn

   1 It would rock if I could view diffs from the web without going via feeds. I envision toggle-style buttons on the recentchanges page, or just links to the CGI, which then displays the diff... --[[madduck]]
   2
   3 > The diffs are actually there, enabled by the [[plugins/recentchangesdiff]]
   4 > plugin, but they are hidden in the XHTML version by the stylesheet.
   5 > You might try a user stylesheet with `div.diff { display: block }`.
   6 > --[[JasonBlevins]]
   7
   8 > > couldn't the diff be displayed as a popup? right now it's too bad because the diff is actually in the page, generated and downloaded, but the user can't see it. I have tried to address the issue by adding stuff to the change.tmpl template, but I may be missing something - and it doesn't quite look right:
   9 > >
  10 > >     --- /usr/share/ikiwiki/templates/change.tmpl    2011-09-05 15:14:19.000000000 -0400
  11 > >     +++ templates/change.tmpl       2011-10-11 13:04:37.704346964 -0400
  12 > >     @@ -39,6 +39,7 @@
  13 > >      </TMPL_LOOP>
  14 > >      </div>
  15 > >      <TMPL_IF DIFF>
  16 > >     +<a href="#" onClick="document.getElementByClass('diff').style = 'block'">[[show diff|wikiicons/diff.png]]</a>
  17 > >      <div class="diff">
  18 > >      <pre>
  19 > >      <TMPL_VAR DIFF>
  20 > >
  21 > > There are a few things wrong with this:
  22 > >
  23 > >  1. I don't like the hardcoded javascript in there, we should use [[plugins/toggle]] or something, but i am not sure how to make the this plugin depend on toggle, or if it is desirable.
  24 > >  2. it doesn't work at all: first it doesn't actually "toggle" and second the javascript somehow gets filtered out of the resulting HTML so we don't even see it
  25 > >  3. if the diffurl parameter is set in the template, we'd actually see two sets of glasses, which is silly. i tried moving the diff button upwards into the PAGES loop, but there the diffurls are file-specific, which also seem quite silly
  26 > >
  27 > > I am looking for guidance on how to improve and fix this now. --[[anarcat]] 2011-10-11
  28
  29 [[!tag wishlist]]