Project

General

Profile

Actions

Defect #5199

closed

diff code coloring using coderay

Added by Patrick Okui over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Time tracking
Target version:
Start date:
2010-03-28
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

This seems not to work when I do pre and code style=diff. CodeRay however can highlight diffs.

As an example:

--- 2010-03-28.gp_invprint    2010-03-28 03:21:46.000000000 +0300
+++ gp_invprint    2010-03-28 03:23:08.000000000 +0300
@@ -535,7 +535,9 @@
     $p->text({align=> 'centre'}, 300, 120, "TAX INVOICE");

     $p->setfont("Arial", "8");
-    $p->text({align=> 'right'}, 580, 120, "TIN:nnnn VAT: mmm");
+    # old style tin. changed due to request.
+    #$p->text({align=> 'right'}, 580, 120, "TIN:nnnn VAT: mmm");
+    $p->text({align=> 'right'}, 580, 120, "e-TIN xxx VAT: rrr");

     $p->setfont($font{"name"}, 10);
     $p->text({align=> 'right'}, 100, 190, "Name: ");
Actions #1

Updated by Jean-Philippe Lang over 14 years ago

  • Category set to Time tracking
  • Status changed from New to Closed
  • Target version set to 1.0.0 (RC)
  • Resolution set to Fixed

This is fixed in r3619.
Note that you have to use class="diff" instead of style="diff". Your example was fixed.

Actions #2

Updated by Holger Just over 14 years ago

  • Status changed from Closed to Reopened

The fix in r3619 breaks the test suite. Specifically test_syntax_highlight(ApplicationHelperTest)

See http://ci.finn.de/builds/1-8-7_redmine-trunk_mysql/3621

Actions #3

Updated by Jean-Philippe Lang over 14 years ago

  • Status changed from Reopened to Closed

Test fixed in r3622.

Actions #4

Updated by Kornelius Kalnbach over 14 years ago

It doesn't look like it's supposed to look. :wrap => :span is not the right way to go, as it doesn't support the new line tokens that diff code needs.

I recommend using .div instead of .html.

Actions

Also available in: Atom PDF