https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292010-03-28T13:54:44ZRedmineRedmine - Defect #5199: diff code coloring using coderayhttps://www.redmine.org/issues/5199?journal_id=155092010-03-28T13:54:44ZJean-Philippe Langjp_lang@yahoo.fr
<ul><li><strong>Category</strong> set to <i>Time tracking</i></li><li><strong>Status</strong> changed from <i>New</i> to <i>Closed</i></li><li><strong>Target version</strong> set to <i>1.0.0 (RC)</i></li><li><strong>Resolution</strong> set to <i>Fixed</i></li></ul><p>This is fixed in <a class="changeset" title="Fixes diff highlighting (#5199)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/3619">r3619</a>.<br />Note that you have to use class="diff" instead of style="diff". Your example was fixed.</p> Redmine - Defect #5199: diff code coloring using coderayhttps://www.redmine.org/issues/5199?journal_id=155102010-03-28T14:14:54ZHolger Just
<ul><li><strong>Status</strong> changed from <i>Closed</i> to <i>Reopened</i></li></ul><p>The fix in <a class="changeset" title="Fixes diff highlighting (#5199)." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/3619">r3619</a> breaks the test suite. Specifically test_syntax_highlight(ApplicationHelperTest)</p>
<p>See <a class="external" href="http://ci.finn.de/builds/1-8-7_redmine-trunk_mysql/3621">http://ci.finn.de/builds/1-8-7_redmine-trunk_mysql/3621</a></p> Redmine - Defect #5199: diff code coloring using coderayhttps://www.redmine.org/issues/5199?journal_id=155122010-03-28T15:20:34ZJean-Philippe Langjp_lang@yahoo.fr
<ul><li><strong>Status</strong> changed from <i>Reopened</i> to <i>Closed</i></li></ul><p>Test fixed in <a class="changeset" title="Fixes broken test." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/3622">r3622</a>.</p> Redmine - Defect #5199: diff code coloring using coderayhttps://www.redmine.org/issues/5199?journal_id=155962010-04-01T18:17:56ZKornelius Kalnbach
<ul></ul><p>It doesn't look like it's <a href="http://coderay.rubychan.de/rays/show/6404" class="external">supposed to look</a>. <code>:wrap => :span</code> is not the right way to go, as it doesn't support the new line tokens that diff code needs.</p>
<p>I recommend using <code>.div</code> instead of <code>.html</code>.</p>