Revision history for DotMGTextSearchExpanded
Revision [18294]
Last edited on 2008-01-28 00:10:59 by JavaWoman [Modified links pointing to docs server]No Differences
Additions:
~&Actually, SafeHTML is needed only for those "parts" of an action where user input **directly** results in HTML output (such as an action parameter //used as// a tag attribute). When HTML is completely **generated** by an action, it's not needed (Wikka should take care by itself that //generated// code is "safe"). --JavaWoman
Revision [6032]
Edited on 2005-02-16 13:55:48 by IanAndolina [Suggested SemanticMarkup code cleanup...]Additions:
----
As part of the SemanticMarkup drive, why not replace //all// that tag-soup table with:
%%(php)
$highlightMatch = preg_replace("/($phrase)/i","<em><em>$1</em></em>",$text,-1);
$matchText = "…".$highlightMatch."…";
$output .= "\n<p>".($i+1)." ".$this->Link($page["tag"])." — ".$page[time]."</p>";
$output .= "\n<blockquote>".$matchText."</blockquote>\n";%%
You remove all the unnecessary table and presentational spans. Use CSS to present the double emphasis exactly how you want, that is what CSS is there for. --IanAndolina
As part of the SemanticMarkup drive, why not replace //all// that tag-soup table with:
%%(php)
$highlightMatch = preg_replace("/($phrase)/i","<em><em>$1</em></em>",$text,-1);
$matchText = "…".$highlightMatch."…";
$output .= "\n<p>".($i+1)." ".$this->Link($page["tag"])." — ".$page[time]."</p>";
$output .= "\n<blockquote>".$matchText."</blockquote>\n";%%
You remove all the unnecessary table and presentational spans. Use CSS to present the double emphasis exactly how you want, that is what CSS is there for. --IanAndolina
