From f4b68599b4677fc3b36d66879f047df20c1105df Mon Sep 17 00:00:00 2001 From: Mathias Date: Fri, 25 Jul 2008 05:02:21 -0400 Subject: [PATCH 1/4] --- doc/sandbox/Test:_with_a_colon_in_its_name.mdwn | 1 + 1 file changed, 1 insertion(+) create mode 100644 doc/sandbox/Test:_with_a_colon_in_its_name.mdwn diff --git a/doc/sandbox/Test:_with_a_colon_in_its_name.mdwn b/doc/sandbox/Test:_with_a_colon_in_its_name.mdwn new file mode 100644 index 000000000..b91d7cb03 --- /dev/null +++ b/doc/sandbox/Test:_with_a_colon_in_its_name.mdwn @@ -0,0 +1 @@ +what happens? From 65f902df434def7979fac6504cbc5d885d6061ec Mon Sep 17 00:00:00 2001 From: Mathias Date: Fri, 25 Jul 2008 05:32:45 -0400 Subject: [PATCH 2/4] --- ...r_blog_items_when_filename_contains_a_colon.mdwn | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn b/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn index 313c1addd..b78515596 100644 --- a/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn +++ b/doc/bugs/No_link_for_blog_items_when_filename_contains_a_colon.mdwn @@ -57,4 +57,15 @@ What do you think about that? Does the patch have any side-effects I didn't see? > I almost really fixed this in 2.53, but missed one case. All fixed now > AFAICS. --[[Joey]] -[[tag done]] +>> Hmm, did you fix it now in 2.54? If so, I suspect there is still one little case left (might well be the last one, +>> at least I hope so ;-) ): I just created a test post in the sandbox here: [[sandbox/test: with a colon in its name]] +>> (btw, why doesn't this get a hyperlink here?). +>> +>> As it is put in the list of blog posts as a relative link, it starts +>> with `` -- this makes the browser think that "test" is a protocol specification which is to replace `http`, +>> so it complains (at least Opera and Firefox/Iceweasel on my Debian Etch do). What I described above for subpages +>> with this name pattern also still happens on my local install (ikiwiki 2.54 on Debian Etch), but this is basically +>> the same problem. +>> +>> I think the cleanest solution would be to quote colons in page names (like it is currently done for slashes)? +>> Starting the links with "`./`", as I proposed above, now seems a bit ugly to me... --Mathias From ac09f1438edde20b2b4b3a6844f93d2a7cfb6bc7 Mon Sep 17 00:00:00 2001 From: "http://ptecza.myopenid.com/" Date: Fri, 25 Jul 2008 08:59:23 -0400 Subject: [PATCH 3/4] * RC1 color plugin --- doc/todo/color_plugin.mdwn | 118 +++++++++++++++++++++++++++++++++++++ 1 file changed, 118 insertions(+) diff --git a/doc/todo/color_plugin.mdwn b/doc/todo/color_plugin.mdwn index cb3e85b74..7b5969f0d 100644 --- a/doc/todo/color_plugin.mdwn +++ b/doc/todo/color_plugin.mdwn @@ -39,3 +39,121 @@ What do you think about it? --[[Paweł|ptecza]] >> doesn't seem to be very hard task. --[[Paweł|ptecza]] >> Yes, it's a good intro plugin, have at it! --[[Joey]] + +--- + +This is a RC1 of my `color` plugin. It works for me well, but all your +comments are very welcome. --[[Paweł|ptecza]] + + --- /dev/null 2008-07-24 09:38:19.000000000 +0200 + +++ color.pm 2008-07-25 14:43:15.000000000 +0200 + @@ -0,0 +1,75 @@ + +#!/usr/bin/perl + +# Ikiwiki text colouring plugin + +# Paweł Tęcza + +package IkiWiki::Plugin::color; + + + +use warnings; + +use strict; + +use IkiWiki 2.00; + + + +sub import { #{{{ + + hook(type => "preprocess", id => "color", call => \&preprocess); + + hook(type => "sanitize", id => "color", call => \&sanitize); + +} #}}} + + + +sub preserve_style(@) { #{{{ + + my ($colors, $text) = @_; + + $colors = '' unless $colors; # foreground and background colors + + $text = '' unless $text; # text + + + + # Check colors + + my ($color1, $color2) = (); + + $colors = lc($colors); # Regexps on lower case strings are simpler + + if ($colors =~ /,/) { + + # Probably defined both foreground and background color + + ($color1, $color2) = ($colors =~ /(.*),(.*)/); + + } + + else { + + # Probably defined only foreground color + + ($color1, $color2) = ($colors, ''); + + } + + + + # Validate colors. Only color name or color code are valid. + + my ($fg, $bg) = (); + + $fg = $color1 if ($color1 && + + ($color1 =~ /^[a-z]+$/ || $color1 =~ /^#[0-9a-f]{3,6}$/)); + + $bg = $color2 if ($color2 && + + ($color2 =~ /^[a-z]+$/ || $color2 =~ /^#[0-9a-f]{3,6}$/)); + + + + my $preserved = ''; + + if ($fg || $bg) { + + $preserved .= 'COLORS {'; + + $preserved .= 'color: '.$fg if ($fg); + + $preserved .= '; ' if ($fg && $bg); + + $preserved .= 'background-color: '.$bg if ($bg); + + $preserved .= '} SROLOC;TEXT {'.$text.'} TXET'; + + } + + + + return $preserved; + + + +} #}}} + + + +sub replace_preserved_style(@) { #{{{ + + my $content = shift; + + + + if ($content) { + + $content =~ s/COLORS {//; + + $content =~ s/} TXET/<\/span>/; + + } + + + + return $content; + +} #}}} + + + +sub preprocess (@) { #{{{ + + return preserve_style($_[0], $_[2]); + +} #}}} + + + +sub sanitize (@) { #{{{ + + my %params = @_; + + + + return replace_preserved_style($params{content}) + + if (exists $params{content}) + +} #}}} + + + +1 + --- /dev/null 2008-07-24 09:38:19.000000000 +0200 + +++ color.mdwn 2008-07-25 14:50:19.000000000 +0200 + @@ -0,0 +1,31 @@ + +\[[!template id=plugin name=color core=0 author="[[Paweł Tęcza|ptecza]]"]] + + + +This plugin can be used to color a piece of text on Ikiwiki page. + +It's possible setting foreground and/or background color of the text. + + + +The plugin syntax is very simple. You only need to type name (e.g. `white`) + +or HTML code of colors (e.g. `#ffffff`) and a text you want to color. + +The colors should by separated using a comma character. + + + +Below are a few examples: + + + + \[[!color white,#ff0000 "White text on red background"]] + + + +Foreground color is defined as a word, background color is defined as HTML + +color code. + + + + \[[!color white "White text on default color background"]] + + + +Foreground color is default color if only one color was typed and a comma + +character is missing. + + + + \[[!color white, "White text on default color background"]] + + + +Background color is missing, so the text is displayed on default background. + + + + \[[!color ,#ff0000 "Default color text on red background"]] + + + +Foreground is missing, so the text has default color. + + + +This plugin is not enabled by default. You can do that in [[ikiwiki.setup]] + +file (hint: `add_plugins` variable). From 9cc28ac18ca946b56e43363623dc258949c89537 Mon Sep 17 00:00:00 2001 From: "http://ptecza.myopenid.com/" Date: Fri, 25 Jul 2008 09:44:24 -0400 Subject: [PATCH 4/4] * Bug report --- doc/bugs/Broken_access_to_Ikiwiki_gitweb.mdwn | 9 +++++++++ 1 file changed, 9 insertions(+) create mode 100644 doc/bugs/Broken_access_to_Ikiwiki_gitweb.mdwn diff --git a/doc/bugs/Broken_access_to_Ikiwiki_gitweb.mdwn b/doc/bugs/Broken_access_to_Ikiwiki_gitweb.mdwn new file mode 100644 index 000000000..f180c778b --- /dev/null +++ b/doc/bugs/Broken_access_to_Ikiwiki_gitweb.mdwn @@ -0,0 +1,9 @@ +I can't check the last changes in Ikiwiki using +[gitweb](http://git.ikiwiki.info/?p=ikiwiki). It looks like XML +validation problem with HTML entity. + +When I click a appropriate link on a [[git]] page, then I can only +see the following error message. --[[Paweł|ptecza]] + +
 
+ -------------------^