http://pear.php.net/bugs/search.php?cmd=display&package_name%5B0%5D=Text_Highlighter Search Results en-us pear-webmaster@lists.php.net pear-webmaster@lists.php.net hourly 1 2000-01-01T12:00+00:00 http://pear.php.net/gifs/pearsmall.gif http://pear.php.net/bugs http://pear.php.net/bugs/16428 Text_Highlighter Feature/Change Request Reported by perthius 2009-07-11T13:44:13+00:00 PHP: 5.2.5 OS: Windows XP Package Version: 0.7.1 Description: ------------ It would be great if the html code generated by this package added a class to the pre element indicating which language is being displayed, eg. <pre class="hl-html"> This would allow me to define different styles for different languages in the same page without having to wrap each code section in another div. .hl_html .hl-code { color: Gray; } .hl_javascript .hl-code { color: Maroon; }]]> Text_Highlighter Feature/Change Request Reported by perthius 2009-07-11T13:44:13+00:00 PHP: 5.2.5 OS: Windows XP Package Version: 0.7.1 Description: ------------ It would be great if the html code generated by this package added a class to the pre element indicating which language is being displayed, eg. <pre class="hl-html"> This would allow me to define different styles for different languages in the same page without having to wrap each code section in another div. .hl_html .hl-code { color: Gray; } .hl_javascript .hl-code { color: Maroon; }]]> 2009-07-13T04:47:50+00:00 perthius &#x61;&#116; lavabit &#x64;&#111;&#x74; com Text_Highlighter Feature/Change Request http://pear.php.net/bugs/14560 Text_Highlighter Bug Reported by thijs 2008-08-25T09:01:46+00:00 PHP: 5.2.0 OS: Linux Package Version: 0.7.1 Description: ------------ When turning on E_STRICT in PHP5 I get these warnings. Could you look into that please? Thanks! Strict Standards: Assigning the return value of new by reference is deprecated in Text/Highlighter.php on line 210 Strict Standards: Assigning the return value of new by reference is deprecated in Text/Highlighter.php on line 364 Strict Standards: Redefining already defined constructor for class Text_Highlighter_Renderer in Text/Highlighter/Renderer.php on line 73]]> Text_Highlighter Bug Reported by thijs 2008-08-25T09:01:46+00:00 PHP: 5.2.0 OS: Linux Package Version: 0.7.1 Description: ------------ When turning on E_STRICT in PHP5 I get these warnings. Could you look into that please? Thanks! Strict Standards: Assigning the return value of new by reference is deprecated in Text/Highlighter.php on line 210 Strict Standards: Assigning the return value of new by reference is deprecated in Text/Highlighter.php on line 364 Strict Standards: Redefining already defined constructor for class Text_Highlighter_Renderer in Text/Highlighter/Renderer.php on line 73]]> 2008-10-09T09:08:59+00:00 thijs &#x61;&#116; debian &#x64;&#111;&#x74; org Text_Highlighter Bug http://pear.php.net/bugs/14480 Text_Highlighter Feature/Change Request Reported by sirzooro 2008-08-09T06:36:42+00:00 PHP: 5.2.6 OS: Windows Package Version: 0.7.1 Description: ------------ <span> generation in Text Highlighter can be improved. Especially these things can be improved (at least for JavaScript): - extra empty <span>s at end of comments; - new line sign is enclosed in <span> too, while it is not needed; - '//' and comment text are in separate spans - can be put in one. Despite these issues page code still validates and looks properly, so I am logging this as enhancement in order to clean up this. Test script: --------------- $code = <<<EOT // comment // comment EOT; $highlighter =& Text_Highlighter::factory('JAVASCRIPT'); echo $highlighter->highlight($code); Expected result: ---------------- <div class="hl-main"><pre><span class="hl-comment">// comment</span> <span class="hl-comment">// comment</span></pre></div> Actual result: -------------- <div class="hl-main"><pre><span class="hl-comment">//</span><span class="hl-comment"> comment</span><span class="hl-comment"></span><span class="hl-code"> </span><span class="hl-comment">//</span><span class="hl-comment"> comment</span><span class="hl-comment"></span></pre></div>]]> Text_Highlighter Feature/Change Request Reported by sirzooro 2008-08-09T06:36:42+00:00 PHP: 5.2.6 OS: Windows Package Version: 0.7.1 Description: ------------ <span> generation in Text Highlighter can be improved. Especially these things can be improved (at least for JavaScript): - extra empty <span>s at end of comments; - new line sign is enclosed in <span> too, while it is not needed; - '//' and comment text are in separate spans - can be put in one. Despite these issues page code still validates and looks properly, so I am logging this as enhancement in order to clean up this. Test script: --------------- $code = <<<EOT // comment // comment EOT; $highlighter =& Text_Highlighter::factory('JAVASCRIPT'); echo $highlighter->highlight($code); Expected result: ---------------- <div class="hl-main"><pre><span class="hl-comment">// comment</span> <span class="hl-comment">// comment</span></pre></div> Actual result: -------------- <div class="hl-main"><pre><span class="hl-comment">//</span><span class="hl-comment"> comment</span><span class="hl-comment"></span><span class="hl-code"> </span><span class="hl-comment">//</span><span class="hl-comment"> comment</span><span class="hl-comment"></span></pre></div>]]> 2008-08-09T06:36:42+00:00 bugzilla &#x61;&#116; poradnik-webmastera &#x64;&#111;&#x74; com Text_Highlighter Feature/Change Request http://pear.php.net/bugs/13350 Text_Highlighter Bug Reported by ussr 2008-03-08T21:13:39+00:00 PHP: 5.2.0 OS: mac os 10.4 Package Version: 0.7.1 Description: ------------ source files using mac-style line braks (\r) are highlighted correctly, but (at least in the HTML-renderer) rendered without linebreaks. This bug is easily fixed by adding a line to the "preprocess" function in Array.php. Mac-style line breaks are normalized by adding: $str = str_replace("\r","\n", $str); I will resend this bug, I am not sure if the previous one was actually submitted. This bug-reporting system is, hm, difficult.]]> Text_Highlighter Bug Reported by ussr 2008-03-08T21:13:39+00:00 PHP: 5.2.0 OS: mac os 10.4 Package Version: 0.7.1 Description: ------------ source files using mac-style line braks (\r) are highlighted correctly, but (at least in the HTML-renderer) rendered without linebreaks. This bug is easily fixed by adding a line to the "preprocess" function in Array.php. Mac-style line breaks are normalized by adding: $str = str_replace("\r","\n", $str); I will resend this bug, I am not sure if the previous one was actually submitted. This bug-reporting system is, hm, difficult.]]> 2008-07-19T04:41:12+00:00 ussr &#x61;&#116; vostok6 &#x64;&#111;&#x74; net Text_Highlighter Bug http://pear.php.net/bugs/12836 Text_Highlighter Feature/Change Request Reported by thijs 2008-01-06T05:01:04+00:00 PHP: Irrelevant OS: Package Version: 0.7.1 Description: ------------ Hi! Thank you for making this class, it's very useful to me. I would also like to include it into Debian Linux as a package. There I encountered a problem with the license (PHP license 3.01): it does not really seem to apply to the package. There's a few references in the license that say that derived versions may not be called PHP, and at the bottom the license claims that this product includes the Zend engine. I do not think that Text Highlighter includes Zend Engine :-) As it seems, the PHP license is written with PHP-the-language in mind. May I suggest a license change to the BSD-license? It seems similar in scope but is simpler, wider known and does not have the problems I outlined above. An alternative is dual licensing under PHP and BSD licenses. thanks!]]> Text_Highlighter Feature/Change Request Reported by thijs 2008-01-06T05:01:04+00:00 PHP: Irrelevant OS: Package Version: 0.7.1 Description: ------------ Hi! Thank you for making this class, it's very useful to me. I would also like to include it into Debian Linux as a package. There I encountered a problem with the license (PHP license 3.01): it does not really seem to apply to the package. There's a few references in the license that say that derived versions may not be called PHP, and at the bottom the license claims that this product includes the Zend engine. I do not think that Text Highlighter includes Zend Engine :-) As it seems, the PHP license is written with PHP-the-language in mind. May I suggest a license change to the BSD-license? It seems similar in scope but is simpler, wider known and does not have the problems I outlined above. An alternative is dual licensing under PHP and BSD licenses. thanks!]]> 2008-08-25T13:04:55+00:00 thijs &#x61;&#116; debian &#x64;&#111;&#x74; org Text_Highlighter Feature/Change Request http://pear.php.net/bugs/12233 Text_Highlighter Feature/Change Request Reported by flippedup 2007-10-12T11:55:59+00:00 PHP: 5.2.4 OS: Linux Package Version: 0.7.0 Description: ------------ This package is absolutely amazing but it lacks in details. I believe that the following configuration options should be allowed: 1.The ability to highlight a line from start to end by prepending the line with a specified character. It would wrap the resulting line with a span and class attribute. 2.The ability to change the default CSS class names that are used for highlighting. 3.The ability to specify whether or not URLs that are found should be linked with an <a href> tag.]]> Text_Highlighter Feature/Change Request Reported by flippedup 2007-10-12T11:55:59+00:00 PHP: 5.2.4 OS: Linux Package Version: 0.7.0 Description: ------------ This package is absolutely amazing but it lacks in details. I believe that the following configuration options should be allowed: 1.The ability to highlight a line from start to end by prepending the line with a specified character. It would wrap the resulting line with a span and class attribute. 2.The ability to change the default CSS class names that are used for highlighting. 3.The ability to specify whether or not URLs that are found should be linked with an <a href> tag.]]> 2007-10-12T15:49:45+00:00 foreground &#x61;&#116; gmail &#x64;&#111;&#x74; com Text_Highlighter Feature/Change Request http://pear.php.net/bugs/11414 Text_Highlighter Feature/Change Request Reported by phpmyforum 2007-06-23T04:20:47+00:00 PHP: 5.2.3 OS: Linux Package Version: Description: ------------ I think it would be nice to have a default-value for highlighter doclinks-option. This should be cached in the Highlighter/*.php files, too. Thanks]]> Text_Highlighter Feature/Change Request Reported by phpmyforum 2007-06-23T04:20:47+00:00 PHP: 5.2.3 OS: Linux Package Version: Description: ------------ I think it would be nice to have a default-value for highlighter doclinks-option. This should be cached in the Highlighter/*.php files, too. Thanks]]> 2007-06-23T04:20:47+00:00 chris &#x61;&#116; phpmyforum &#x64;&#111;&#x74; de Text_Highlighter Feature/Change Request http://pear.php.net/bugs/11172 Text_Highlighter Feature/Change Request Reported by sirzooro 2007-05-29T01:02:24+00:00 PHP: Irrelevant OS: N/A Package Version: 0.6.9 Description: ------------ Currently highlighted code is wrapped with <div class="hl-main"><pre>...</pre></div>. Add option to use <code class="hl-main">...</code> instead. Trere is also one more thing to consider - new line handling. Consider adding new options 'new line handling' with following values: - none (left them as-is); - html (replace with <br>); - xhtml (replace with <br />).]]> Text_Highlighter Feature/Change Request Reported by sirzooro 2007-05-29T01:02:24+00:00 PHP: Irrelevant OS: N/A Package Version: 0.6.9 Description: ------------ Currently highlighted code is wrapped with <div class="hl-main"><pre>...</pre></div>. Add option to use <code class="hl-main">...</code> instead. Trere is also one more thing to consider - new line handling. Consider adding new options 'new line handling' with following values: - none (left them as-is); - html (replace with <br>); - xhtml (replace with <br />).]]> 2007-10-25T15:39:54+00:00 bugzilla &#x61;&#116; poradnik-webmastera &#x64;&#111;&#x74; com Text_Highlighter Feature/Change Request http://pear.php.net/bugs/11168 Text_Highlighter Feature/Change Request Reported by sirzooro 2007-05-28T15:14:31+00:00 PHP: 4.4.5 OS: windows Package Version: 0.6.9 Description: ------------ PHP highlighting doesn't work at all Test script: --------------- require_once 'Text/Highlighter.php'; $highlighter =& Text_Highlighter::factory('PHP'); echo $highlighter->highlight('print "abc"; // print "abc"'); Expected result: ---------------- Highlighted PHP code sample Actual result: -------------- <div class="hl-main"><pre><span class="hl-code">print &quot;abc&quot;; // print &quot;abc&quot;</span></pre></div>]]> Text_Highlighter Feature/Change Request Reported by sirzooro 2007-05-28T15:14:31+00:00 PHP: 4.4.5 OS: windows Package Version: 0.6.9 Description: ------------ PHP highlighting doesn't work at all Test script: --------------- require_once 'Text/Highlighter.php'; $highlighter =& Text_Highlighter::factory('PHP'); echo $highlighter->highlight('print "abc"; // print "abc"'); Expected result: ---------------- Highlighted PHP code sample Actual result: -------------- <div class="hl-main"><pre><span class="hl-code">print &quot;abc&quot;; // print &quot;abc&quot;</span></pre></div>]]> 2007-05-28T16:32:45+00:00 bugzilla &#x61;&#116; poradnik-webmastera &#x64;&#111;&#x74; com Text_Highlighter Feature/Change Request