From 199411698a2c4dba705da1ce3bf31cd2abb48395 Mon Sep 17 00:00:00 2001 From: "https://id.koumbit.net/anarcat" Date: Thu, 19 Jul 2012 10:40:28 -0400 Subject: [PATCH 1/2] fixed all issues here --- doc/todo/provide_inline_diffs_in_recentchanges.mdwn | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/doc/todo/provide_inline_diffs_in_recentchanges.mdwn b/doc/todo/provide_inline_diffs_in_recentchanges.mdwn index 7e95791c6..98e8663fa 100644 --- a/doc/todo/provide_inline_diffs_in_recentchanges.mdwn +++ b/doc/todo/provide_inline_diffs_in_recentchanges.mdwn @@ -11,13 +11,15 @@ It would rock if I could view diffs from the web without going via feeds. I envi > > > > Unfortunately it has some issues: > > -> > 1. it assumes the toggle.js code is loaded somehow -> > 2. if the toggle code isn't loaded the diffs are displayed (which is arguably better than showing nothing since we ship the diff to the UA anyways...) +> > 1. it assumes the toggle.js code is loaded somehow - now loaded manually +> > 2. if the toggle code isn't loaded the diffs are displayed (which is arguably better than showing nothing since we ship the diff to the UA anyways...) - i actually think that's fine > > 3. it will show only if there's a revert URL, which is backwards, but otherwise the display is weird, with each button on its own line fixed! -> > 4. if the diffurl parameter is set in the template, we'd actually see two sets of glasses, which is silly. +> > 4. if the diffurl parameter is set in the template, we'd actually see two sets of glasses, which is silly. - just added a tmp_unless to fix this. > > > > I feel this should nevertheless be implemented because if we're going to compile all this crap in the page anyways and send it to the client, why not allow the user to show it? I also feel that showing it by default is a lesser evil for non-javascript users. > > > > -- [[anarcat]] 2012-03-03 +> > > I have pushed a new version of this patch to my branch, which fixes all the above issues. I think this is ready to be merged now. -- [[anarcat]] 2012-07-19 + [[!tag wishlist patch]] From 9fb6451ef94db59442c5b3867516baecdfcee01d Mon Sep 17 00:00:00 2001 From: "https://id.koumbit.net/anarcat" Date: Thu, 19 Jul 2012 12:49:12 -0400 Subject: [PATCH 2/2] better listing of the issues i am interested in --- doc/users/anarcat.mdwn | 30 ++++++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/doc/users/anarcat.mdwn b/doc/users/anarcat.mdwn index d868d7493..a62c741cd 100644 --- a/doc/users/anarcat.mdwn +++ b/doc/users/anarcat.mdwn @@ -1 +1,31 @@ See + +[[!toc]] + +My todos +======== + +... or the ones I commented it, to be more precise. + +[[!inline pages="todo/* and !todo/done and !link(todo/done) and +link(users/anarcat) and !todo/*/*" feeds=no actions=yes archive=yes show=0]] + +Done +---- + +[[!inline pages="todo/* and !todo/done and link(todo/done) and +link(users/anarcat) and !todo/*/*" feeds=no actions=yes archive=yes show=0]] + +My bugs +======= + +... same. + +[[!inline pages="bugs/* and !bugs/done and !link(bugs/done) and +link(users/anarcat) and !bugs/*/*" feeds=no actions=yes archive=yes show=0]] + +Fixed +----- + +[[!inline pages="bugs/* and !bugs/done and link(bugs/done) and +link(users/anarcat) and !bugs/*/*" feeds=no actions=yes archive=yes show=0]]