Edgewall Software
Modify

Opened 16 years ago

Closed 16 years ago

Last modified 8 years ago

#2078 closed enhancement (fixed)

Allow anchors in changeset traclinks

Reported by: anonymous Owned by: Christian Boos
Priority: low Milestone: 0.11
Component: version control/changeset view Version: devel
Severity: minor Keywords: traclinks
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

When referencing large changesets, it would be really useful if one could enter for example [2177#file17] to link to http://projects.edgewall.com/trac/changeset/2177#file17

Attachments (0)

Change History (8)

comment:1 by Christian Boos, 16 years ago

Milestone: 0.9
Owner: changed from Jonas Borgström to Christian Boos
Severity: trivialminor

This is a good idea, along with the general idea that the #xxx suffix should be used to refer to a particular detail in the linked object, as it's already the case for Wiki links and was recently suggested for source links (see #2011).

Alternatively, The TracDiff support the notion of restricted changeset (i.e. the changeset information restricted to what happened on a given path), which support the syntax [1234/trunk/README], since r… (wait, committing first…) since r2256.

comment:2 by Christian Boos, 16 years ago

Depends on #2168.

There's one caveat: the generation of the #fileXYZ sequence should not depend on the diff options (currently, a file containing only white space changes will simply dissappear from the list if the ignore white-space changes option is checked)

comment:3 by Christian Boos, 16 years ago

Milestone: 0.10
Resolution: fixed
Status: newclosed

I think that the restricted changeset TracLinks (i.e. [1234/trunk/README]) are way better for this ticket's use case than the suggested syntax.

First, as already mentioned, the number of files shown is susceptible to change depending on the Diff options. Then, refering to a particular anchor within the full changeset is not going to help for performance when trying to view a large changeset.

comment:4 by Christian Boos, 16 years ago

In fact, when changing the diff options, only what gets actually displayed can vary, but the #fileXYZ sequence never changes. So this might actually get implemented as part of #2168.

comment:5 by Christian Boos, 15 years ago

Component: generalchangeset view
Keywords: TracLinks added
Milestone: 0.100.11

Original request for enhancement implemented in r4367.

Don't forget that the first entry is #file0! But anyway, there's now an anchor which makes it easy to pick the right number without having to count ;)

comment:6 by Ryan J Ollos, 8 years ago

Keywords: traclinks added; TracLinks removed

comment:7 by Ryan J Ollos, 8 years ago

Keywords: traclink added; traclinks removed

comment:8 by Ryan J Ollos, 8 years ago

Keywords: traclinks added; traclink removed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


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