The searchhighlight plugin insert a <span> around all terms found in the page. It should not do that inside some tags :
- script : javascript parameters are broken
- mailto : email addresses are broken
...

I forgot the P1.html page. For me it is ok to use it.
But it is more complex. Without rewriting it should be index.php?/archives/P1.html.

judebert wrote:I just commited this to the 1.1 baseline! Thanks for the patch!

Incidentally, it looks like you're using 1.0;

Thanks for applying it !
Yes I'm using 1.0 in production. And I'm too lazy to install a test 1.1...

I post a diff for the "all categories" url. This is always the "?frontpage" with or without static page. When a static page is made the front page, it is "/" or "index.php" which points to this static page. Then "all categories" whithout the patch is invalid because it points to the static page. I ...

When using static pages the frontpage is not the all categories page. This page is accessible with <index url>?frontpage

diff include/plugin_internal.inc.php.orig include/plugin_internal.inc.php
1514c1514
< $serendipity['serendipityHTTPPath'] . $serendipity['indexFile'],
---
> $serendipity ...

In fact I used s9y without problem. But I've recently tried to validate my site with ht//check which see all [] as badly formatted url. Of course there is some more [] (cview, etc) in s9y but this one is easy to correct.

Here is a diff to correct some mistakes. Note that in French signs containing 2 symbols (;:!?) need a space before and a space after.

http://www.thalix.com/documents/diff

diff functions_comments.inc.php.orig functions_comments.inc.php
260c260
< $comment['email'] = htmlspecialchars(str_replace('@', '[at]', $comment['email']));
---
> $comment['email'] = htmlspecialchars(str_replace('@', '%5Bat%5D', $comment['email']));

[] are forbidden in w3c recommandations