Closed Bug 98308 Opened 23 years ago Closed 23 years ago

Bugzilla should support <A name="42"> for comments

Categories

(Bugzilla :: User Interface, defect, P3)

2.15
defect

Tracking

()

VERIFIED DUPLICATE of bug 71840

People

(Reporter: gerv, Assigned: gerv)

Details

Attachments

(1 file)

Bugzilla's comments should be numbered and you should be able to do:
http://bugzilla.mozilla.org/show_bug.cgi?id=12345#6
to get to comment number 6.

Additionally, the string "comment <num>" should be hotlinked by the same code
that does URLs and bug numbers.

Gerv
Priority: -- → P3
Target Milestone: --- → Bugzilla 2.16
Ah, so I'm not the only one to have thought of this. :-)

I suggest the Bugzilla syntax "bug 98308.2001-09-07.1" (and also "comment 
98308.2001-09-07.1") for linking to a particular comment (that example would 
link to this comment itself), since it's easier to count the number of comments 
on a particular day than to count the total number of comments. ("Comment
2001-09-07.1" should also work within a bug, and bug/comment nnnnn.yyyy-mm-dd 
should be a synonym for bug/comment nnnnn.yyyy-mm-dd.1.)
You wouldn't need to count comments because each comment would have its number
next to it. So we can have a simple numbering scheme from 1 to N.

However, I like the idea of hyperlinking bug 34567#8 for comment 8 on bug 34567. 

This bug is a dupe, BTW.

Gerv
Ok, the following code adds comment numbers, incrementing from 0 (Description)
for each bug, with little links at the end of the "Additional Comments" string.
It also linkifies text of the following form:

Within a bug:
comment 1
comment #1
com 1
(basically, the same regexp as bug, but with com(ment)? instead.)

Between bugs:
bug 30000#1
bug #30000 #4.
bug 30000 comment 1
bug 30000, com 4
etc.

but _not_:
bug 30000, #5
bug #30000, #6
because I think this would get the wrong result some times.

Gerv
Status: NEW → ASSIGNED
Attached patch Patch v.1Splinter Review
Lest you think I'm ignoring the patch, the method you're using to create the
reference IDs has already been shot down by Myk in bug 71840.  Rather than
continuing to have 2 bugs, you can upload the fixed copy to the original bug, or
update Jake's last patch on that bug (which has the corrections Myk asked for)
to add the hyperlinkifying stuff.  Although I still think the numbering system
is up for debate in case someone deletes a comment...

*** This bug has been marked as a duplicate of 71840 ***
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
clearing milestone for invalid/wontfix/worksforme/duplicate so they'll show up
as untriaged if they get reopened.
Target Milestone: Bugzilla 2.16 → ---
vrfy dupe, bug's fixed :)
Status: RESOLVED → VERIFIED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: