Closed Bug 783091 Opened 7 years ago Closed 7 years ago

Missing links when there are multiple 'review?(foo)' shown in attachment box on show_bug.cgi

Categories

(bugzilla.mozilla.org Graveyard :: Extensions: InlineHistory, defect)

Production
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: glob)

Details

Attachments

(1 file)

Attached image Screenshot
I did the following:
1) Visited https://bugzilla.mozilla.org/show_bug.cgi?id=764894
2) Used 'Add an attachment'
3) Chose a file, ticked 'patch', added summary/description, set review to '?' and added three email addresses comma separated.
4) Submitted the form
5) Returned to the updated bug

Expected:
Three 'review?(foo)' entries in the attachment box on show_bug.cgi, all of which are links that jump to the appropraite bug comment.

Actual:
As expected, except the middle of the three 'review?(foo' strings is not a link. (See screenshot).
Component: General → Extensions: Inline History
Clarifying summary.
Summary: Three pending reviews 'review?(foo)' shown in attachment box on show_bug.cgi, middle one not linkified → Missing links when there are multiple 'review?(foo)' shown in attachment box on show_bug.cgi
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.0/
modified extensions/InlineHistory/template/en/default/hook/bug/comments-aftercomments.html.tmpl
modified extensions/InlineHistory/web/inline-history.js
Committed revision 8336.

Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/
modified extensions/InlineHistory/template/en/default/hook/bug/comments-aftercomments.html.tmpl
modified extensions/InlineHistory/web/inline-history.js
Committed revision 8355.
Assignee: nobody → glob
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Awesome, thank you! :-D
Product: bugzilla.mozilla.org → bugzilla.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.