]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/add_remove_to_actionlist.mdwn
Added a comment
[git.ikiwiki.info.git] / doc / todo / add_remove_to_actionlist.mdwn
index 92beabc94b442bdb95a3fb10e610b1050e085cca..b50fe885fde7d858852d5e8dfb2c23bd0dbb814b 100644 (file)
@@ -15,68 +15,6 @@ I haven't begun on the 'rename' plugin. -- [[Jon]]
 
 [[!tag wishlist patch]]
 
-> This seems like weird overloading of the header parameter - it's
-> table data, except when it isn't.
-
-> > My first cut (now rebased out of existence I think) introduced a
-> > new "headerblock" parameter, but trying to clearly document the
-> > interaction of data/headerblock/header parameters was too awkward. -- [[Jon]]
-
-> Perhaps
-> something like this would be easier to use in practice?
-> (and also more featureful :-) )
->
->     \[[!table header="2 rows 1 column" data="""
->     Name | Platform ||
->     | Windows | Mac | Linux
->     ikiwiki | no | yes | yes
->     Starcraft | yes | yes | via Wine
->     """]]
-
-> > Thanks for your prompt feedback!
-> > 
-> > This would probably be good, yes, and having mixed row/column headers is
-> > definitely a nice-to-have. I don't relish the prospect of writing the parser
-> > but I see you've made a stab already...
-> > 
-> > One thing you'd lose, but it's debatable whether this is valuable, would be
-> > to have the header defined in the directive, and the remaining table data
-> > declared in an external CSV. -- [[Jon]]
-
-> intended to be rendered like
->
-> <table>
-> <tr><th>Name</th><th colspan=2>Platform</th>
-> <tr><th></th><th>Windows</th><th>Mac</th><th>Linux</th></tr>
-> <tr><th>ikiwiki</th><td>no</td><td>yes</td><td>yes</td></tr>
-> <tr><th>Starcraft</th><td>yes</td><td>yes</td><td>via Wine</td></tr>
-> </table>
->
-> (Deliberately switching to plain-text to make it more obvious
-> what's a `<th>` and what's `<td>`.)
->
-> Vague pseudocode for parsing `headers`
-> (possibly even valid Perl, I'm not sure):
->
->     my ($header_rows, $header_cols);
->     while ($header =~ s/(\d*)\W*(\w+)//) {
->         my $n = ($1 or 0);
->         my $what = $2;
->         if ($what =~ m/rows?/) {
->             $header_rows = $n;
->         }
->         elif ($what =~ m/col(?:umn)?s?/) {
->             $header_cols = $n;
->         }
->     }
->
-> and it would even be fairly easy to extend to support
-> `(first|last|)\W*(\d*)\W*(\w+)` later, e.g.
-> `header="1 row, first 2 cols, last column"`.
->
-> --[[smcv]]
-
-> > To be clear I think your suggestion is a good one, but my hack has
-> > addressed my immediate need so it's the one I'm deploying at $ork for the
-> > time being. I'm unlikely to have time to implement this solution in the
-> > near future. -- [[Jon]]
+> I accidentally pushed an incomplete patch to that branch that starts the
+> work of doing the same for rename, but it's not working yet, to merge one
+> would need to cherry-pick the other patches for now. Sorry. -- [[Jon]]