Project

General

Profile

Actions

Feature #2715

closed

"Magic links" to notes

Added by Ве Fio almost 16 years ago. Updated over 12 years ago.

Status:
Closed
Priority:
Normal
Category:
Text formatting
Target version:
Start date:
2009-02-11
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

This was part of Feature #391, but the issue was closed.
e.g. #391.1 -> http://www.redmine.org/issues/show/391#note-1

Edit:

The syntax implemented in Redmine for 1.4.0 is:

#2715#note-41

or the short version:

#2715-41

Files

issue_note_link.patch (4.39 KB) issue_note_link.patch v2 Etienne Massip, 2011-04-09 15:03

Related issues

Related to Redmine - Feature #391: Links to notes/comments/replies in issues.Closed

Actions
Related to Redmine - Feature #12912: Issue-notes Redmine links: append actual note reference to rendered linksClosed

Actions
Has duplicate Redmine - Feature #6368: Create a syntax for links to commentsClosed2010-09-12

Actions
Has duplicate Redmine - Feature #5326: Wiki shortcuts for linking directly to issue comments?Closed2010-04-15

Actions
Actions #1

Updated by threexk threexk almost 15 years ago

I'm also interested in this feature. It would be handy to have a simple way to refer to a note on the current issue other than writing "see note 24" or making a full URL link.

Actions #2

Updated by Eric Davis almost 15 years ago

  • Category set to Text formatting
Actions #3

Updated by Guilherme Schneider over 14 years ago

+1

Actions #4

Updated by Fernando Hartmann over 14 years ago

+1

Actions #5

Updated by Burt Culver about 14 years ago

+1

Actions #6

Updated by Martin Hartung about 14 years ago

+1

Actions #7

Updated by Hajime Fujita about 14 years ago

+1

Actions #8

Updated by Lucas CORBEAUX almost 14 years ago

+1

Actions #9

Updated by Max Rozenoer almost 14 years ago

+1. I was used to this in Trac :)

Actions #10

Updated by Etienne Massip over 13 years ago

  • File issue_note_link.patch added

threexk threexk wrote:

I'm also interested in this feature. It would be handy to have a simple way to refer to a note on the current issue other than writing "see note 24" or making a full URL link.

I agree this could be very useful.

Hope we could include this soon, here's a patch.

Actions #11

Updated by Etienne Massip over 13 years ago

  • Target version set to Candidate for next major release
Actions #12

Updated by Etienne Massip over 13 years ago

  • File deleted (issue_note_link.patch)
Actions #13

Updated by Etienne Massip over 13 years ago

Cleaner patch with test.

All test pass.

Syntax is #2715-14 for note 14 of issue #2715.

Actions #14

Updated by Gilles Cornu over 13 years ago

+1

Actions #15

Updated by Martin Denizet (redmine.org team member) over 13 years ago

I think it's a must have.
My users use #<note-id> to make references to notes which actually makes links to issues.
It creates a lot of confusion.
I agree with the syntax #2715-14, thank you Etienne for the patch!

Actions #16

Updated by Ling Li over 13 years ago

I tested the patch from Etienne Massip on 1.1.2 and (with some manual fiddle to fix the failed patching) it works well. Thanks!

While we are at this, can we add yet another syntax to refer to notes within the same issue? I.e.,

  • note:5 refers to note 5 of the same issue. I used ":" instead of "-" since the former is in the spirit of "commit:f30e13e43" and "source:some/file"
  • #2715-14 (or maybe we should use #2715:14 for consistency) refers to note 14 of issue #2715.

What do you think?

Actions #17

Updated by Ling Li over 13 years ago

Or, if people like the syntax to be compact/terse:
  • #:5 refers to note 5 of the same issue
  • #2715:14 refers to note 5 of issue #2715.
Actions #18

Updated by Etienne Massip over 13 years ago

I like the last one. What about a period or a 'n' instead of a semicolon ?

That is :

  • #.5 refers to note 5 of the same issue
  • #2715.14 refers to note 5 of issue #2715

or

  • #n5 refers to note 5 of the same issue
  • #2715n14 refers to note 5 of issue #2715.
Actions #19

Updated by Ling Li over 13 years ago

I see a period or an 'n' makes the parsing easier than a colon ':'. I like the period version (now we kinda get back to the initial suggestion in Description).

Actions #20

Updated by Terence Mill over 13 years ago

+1

Actions #21

Updated by Leonids Maslovs over 13 years ago

+1

Actions #22

Updated by Frotran Def over 13 years ago

+1

Actions #23

Updated by Colin Mollenhour about 13 years ago

+1

Fyi, the patch applied cleanly (except for the unit tests) to my installation of 1.2.1 and works wonderfully. Thanks Etienne!

Actions #24

Updated by Dmitry Salashnik about 13 years ago

+1

I also think period is better separator :)

Actions #25

Updated by Vagner Santos about 13 years ago

+1

Actions #26

Updated by Anton Lukashev about 13 years ago

+1

Actions #27

Updated by Nayuta Taga about 13 years ago

+1

Actions #28

Updated by Nick Bolton about 13 years ago

Actions #29

Updated by Sergei Danilov almost 13 years ago

+1
It's easy way to link timelog comment with issue note

Actions #30

Updated by ohcibi ohcibi almost 13 years ago

+1

Actions #31

Updated by Daniel Ritz almost 13 years ago

+1

Etienne Massip wrote:

  • #n5 refers to note 5 of the same issue
  • #2715n14 refers to note 5 of issue #2715.

That's the format I like the most.

Actions #32

Updated by Stéphane Gourichon almost 13 years ago

+1 whatever the syntax finally kept (letter, punctuation etc).

Why 'n' ? If a letter at all, since it's a link to a comment 'c' seems more natural than 'n', e.g. #2715c14 .

Actions #33

Updated by pasquale [:dedalus] almost 13 years ago

Stéphane Gourichon wrote:

#2715c14 .

+1: behaves as bugzilla

Actions #34

Updated by Jean-Philippe Lang almost 13 years ago

  • Assignee set to Jean-Philippe Lang
  • Target version changed from Candidate for next major release to 1.4.0

Everyone is OK with #2715c14?

Actions #35

Updated by Martin Hartung almost 13 years ago

Lacking Ruby skills I did not check the patch itself, but am very much ok with the suggested synax.
The original example was #2715-14 and that's how it should be in my point of view.

Actions #36

Updated by Etienne Massip almost 13 years ago

Jean-Philippe Lang wrote:

Everyone is OK with #2715c14?

What about the use of a generic #<issue number>#<html anchor name> which would allow to refer to any anchor in the page (like for references to wiki pages), including #note-<note number>?

Actions #37

Updated by Jean-Philippe Lang almost 13 years ago

Actually, I don't really like #2715c14 much. I would prefer #2715-14 or even #2715:14. But I agree that #2715#note-14 makes a lot of sense, it's just that it's less compact.

Actions #38

Updated by Etienne Massip almost 13 years ago

Jean-Philippe Lang wrote:

Actually, I don't really like #2715c14 much.

Agree, the c is not readable; what would be great as it has been suggested is a syntax which is also usable to refer to a note in this issue, e.g. #-39 for this note.

#2715#note-14 (or #2715/note-14, #2715:note-14, etc.) has the merit that it is understood when reading it that it refers to a specific comment of a specific issue and not anything else related to the issue. That would make #note-14 the link to the in-issue note 14, which seems great too.

Actions #39

Updated by Simon Edwards almost 13 years ago

Etienne Massip wrote:

#2715#note-14 (or #2715/note-14, #2715:note-14, etc.) has the merit that it is understood when reading it that it refers to a specific comment of a specific issue and not anything else related to the issue. That would make #note-14 the link to the in-issue note 14, which seems great too.

I agree that this syntax does make the most sense, even though it's less compact. Having the links within the issue being #note-n agrees with the current anchor format, so it's the most intuitive method. This approach gets my vote.

Actions #40

Updated by Gilles Cornu almost 13 years ago

Since there is so much different tastes around, why not:
  • support more than a unique separator pattern (:,-,.,c,n,-note,...).
    • Personally, we deployed the patch with '-' syntax (e.g. #2715-41), and I find it quite nice, intuitive and convenient
  • make the pattern(s) configurable/overridable in Redmine settings ?
    • (okay, it's maybe overkill feature, but it has the advantage to stop this discussion ;-)

I don't think there is any drawback to support more than one possible syntax. Is there any?

Actions #41

Updated by Jean-Philippe Lang almost 13 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

Feature is added in r8877 and accepts the 2 syntaxes:

#2715#note-41

or the short version:

#2715-41

I don't think it's a good thing to make it configurable. This should be the same across all Redmine instances.

Actions #42

Updated by Jean-Philippe Lang almost 13 years ago

  • Description updated (diff)
Actions

Also available in: Atom PDF