From 8a45362d55a4b563764f997d7419623a17da86ba Mon Sep 17 00:00:00 2001 From: "http://ptecza.myopenid.com/" Date: Sat, 26 Jul 2008 09:42:27 -0400 Subject: [PATCH] * Response --- doc/todo/color_plugin.mdwn | 29 +++++++++++++++++++++++++---- 1 file changed, 25 insertions(+), 4 deletions(-) diff --git a/doc/todo/color_plugin.mdwn b/doc/todo/color_plugin.mdwn index fa460a386..b82e0b704 100644 --- a/doc/todo/color_plugin.mdwn +++ b/doc/todo/color_plugin.mdwn @@ -46,19 +46,37 @@ This is a RC1 of my `color` plugin. It works for me well, but all your comments are very welcome. --[[Paweł|ptecza]] > Sure, I have a couple. -> + +>> Great! Thank you very much! --[[Paweł|ptecza]] + > The preprocess function is passed named parameters. The hack you have of > hardcoding use of `$_[0]` and `$_[2]` can fail at any time. -> + +>> But the problem is that arguments of my plugin don't have a name. +>> How can I identify them in `params` hash? + +>> Similar hardcoded method I've found in `img` plugin :) But only one +>> argument is not named there (image path). + +>> Maybe I shouldn't use so simple plugin syntax? For following syntax +>> I wouldn't have that problem: + +>> \[[!color fg=white bg=red text="White text on red background"]] + > `replace_preserved_style` is passed a single parameter, so its prototype > should be `($)`, not `(@)`. Ditt `preserve_style`, it should have > `($$)`. -> + +>> OK, it will be fixed. + > The sanitize hook is always passed `$params{content}`, so there should be > no reason to check that it exists. Also, it shouldn't be done in a > sanitize hook, since html sanitization could run _after_ that santize > hook. It should use a format hook. -> + +>> Probably you're right. It was rather paranoid checking ;) Thanks for +>> the hook hint! + > The preprocess hook needs to call `IkiWiki::preprocess` on the content > passed into it if you want to support nesting other preprocessor > directives inside the color directive. See `preprocess_toggleable` in the @@ -68,6 +86,9 @@ comments are very welcome. --[[Paweł|ptecza]] > The method used by toggle of using two real `
`s seems slightly > better. --[[Joey]] +>> I don't like that too, but I didn't have better idea :) Thank you for +>> the hint! I'll take a look at `toggle` plugin. + --- /dev/null 2008-07-24 09:38:19.000000000 +0200 +++ color.pm 2008-07-25 14:43:15.000000000 +0200 @@ -0,0 +1,75 @@