Project

General

Profile

Actions

Defect #7495

closed

Python multiline comments highlighting problem in Repository browser

Added by Graham King about 13 years ago. Updated almost 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Third-party libraries
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

When viewing Python code (from an SVN repo) in the repository viewer multiline PyDoc style comments aren't highlighted correctly.

For example, the multiline comment in this ticket will highlight correctly, but not in the source viewer.

#!/usr/bin/python

def foo(bar):
    """Takes a bar and converts to a foo.
    This is not a good idea.""" 
    return bar

In the repository source viewer only the first line is highlighted.

We're running Redmine trunk r4786. I'm using Chrome on Linux.

About your application's environment
Ruby version              1.8.7 (x86_64-linux)
RubyGems version          1.3.5
Rack version              1.0
Rails version             2.3.5
Active Record version     2.3.5
Active Resource version   2.3.5
Action Mailer version     2.3.5
Active Support version    2.3.5
Application root          /home/redmine/redmine
Environment               production
Database adapter          mysql
Database schema version   20101114115359

Related issues

Related to Redmine - Feature #4264: Update CodeRay to 1.0 finalClosed

Actions
Actions #1

Updated by Ве Fio almost 13 years ago

This actually happens to me also. I was wondering why it wasn't highlighting the multiline strings correctly.

Actions #2

Updated by Etienne Massip almost 13 years ago

It's a CodeRay issue which has already been fixed in trunk. Next release, due in 16 days, (1.0 RC1 !) should include the fix.

Actions #3

Updated by Mischa The Evil almost 13 years ago

  • Category changed from Text formatting to Third-party libraries
  • Target version set to Unplanned backlogs
  • Start date deleted (2011-01-30)
  • Affected version (unused) set to devel

Etienne Massip wrote:

It's a CodeRay issue which has already been fixed in trunk [...]

I presume you are talking about the trunk in the SCM of CodeRay, thus not Redmine trunk?!

Etienne Massip wrote:

Next release, due in 16 days, (1.0 RC1 !) should include the fix.

Since CodeRay 1.0 final might not make in into Redmine 1.3.0 I've created issue #8255 by copying issue #4264, changed issue-relations to reflect the upstream roadmap:

- related to Feature #4264: Update CodeRay to 1.0 final
+ related to Feature #8255: Consider updating CodeRay to 1.0 RC-x

and changed to new - more appropriate - category...

Actions #4

Updated by Etienne Massip almost 13 years ago

Mischa The Evil wrote:

I presume you are talking about the trunk in the SCM of CodeRay, thus not Redmine trunk?!

Indeed, sorry.

Since CodeRay 1.0 final might not make in into Redmine 1.3.0 I've created issue #8255 by copying issue #4264, changed issue-relations to reflect the upstream roadmap:

- related to Feature #4264: Update CodeRay to 1.0 final
+ related to Feature #8255: Consider updating CodeRay to 1.0 RC-x

and changed to new - more appropriate - category...

I just changed the relation to #8255 as a "blocked by", if it is ok for you ?

Actions #5

Updated by Etienne Massip over 12 years ago

Actually it is not a CodeRay issue, it's a Redmine issue which is not easy to fix since it is due to RM slicing the syntax highlighter's output to insert line numbering, breaking the HTML tree and thus the style inheritance.

And if we rely on CodeRay line numbering as suggested by #3382, then I guess this will definitely deny line wrapping, which may be annoying when viewing code attachments.

Any thoughts?

Actions #6

Updated by Etienne Massip over 12 years ago

Blocked by CodeRay issue 222.

Actions #7

Updated by Etienne Massip almost 12 years ago

  • Status changed from New to Resolved
  • Target version changed from Unplanned backlogs to 1.4.0
  • Affected version (unused) changed from devel to 1.3.2
  • Resolution set to Fixed
  • Affected version set to 1.3.2

Fixed with r9389 but not merged. Part of 1.4.0?

Actions #8

Updated by Jean-Philippe Lang almost 12 years ago

  • Status changed from Resolved to Closed

Merged, thanks.

Actions

Also available in: Atom PDF