Edgewall Software
Modify

Opened 14 years ago

Last modified 8 years ago

#8961 new enhancement

[PATCH] Linkable lines on diff view

Reported by: niko.sten@… Owned by:
Priority: normal Milestone: next-major-releases
Component: wiki system Version: 0.12dev
Severity: minor Keywords: patch diff lines link needfixup
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Lines are not linkable in diff view. This patch modifies diff_div.html so that lines are linkable.

Attachments (1)

linked_diff_lines.diff (12.5 KB ) - added by anonymous 14 years ago.
Proposed patch to add linkable lines in diff view

Download all attachments as: .zip

Change History (10)

by anonymous, 14 years ago

Attachment: linked_diff_lines.diff added

Proposed patch to add linkable lines in diff view

comment:1 by Remy Blank, 14 years ago

Milestone: 0.12next-minor-0.12.x

Nice idea. Can you describe how the anchor names are constructed?

comment:2 by anonymous, 14 years ago

BL1 would mean line number 1 in the base file while CL1 means line 1 in the changed file.

comment:3 by Christian Boos, 14 years ago

Milestone: next-minor-0.12.x0.13
Priority: lownormal

Nice patch.

It will also certainly be useful for patch comments (#2035).

comment:4 by Christian Boos, 14 years ago

Keywords: needfixup added

We need to support multiple files per patch.

For the same feature, GitHub uses #Lm[LR]n (m number of the file, Left or Right, n number of the line) and BitBucket uses #chg-<path>_oldline<n>.

comment:5 by Christian Boos, 13 years ago

Milestone: 0.130.14

comment:6 by Christian Boos, 13 years ago

Owner: set to Christian Boos

comment:7 by Ryan J Ollos, 9 years ago

Milestone: next-dev-1.1.xnext-major-releases

Retargetting tickets to narrow focus for milestone:1.2. Please move the ticket back to milestone:next-dev-1.1.x if you intend to resolve it by milestone:1.2.

comment:8 by Ryan J Ollos, 9 years ago

Owner: Christian Boos removed

comment:9 by figaro, 8 years ago

Keywords: patch added

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.