726181 - Shoud linkify URIs in Splinter

Status: RESOLVED INCOMPLETE

(bugzilla.mozilla.org :: Splinter, enhancement, P5)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

If I paste a link into a line's comment, would be nice to have that linkified in splinter.

Comment 1

[David Lawrence [:dkl]]

FWIW, Splinter is just constructing a custom comment to add to the bug comments when you publish the review. Whatever links you have in the review comments of Splinter will be linkified when someone sees the review in the bug comments themselves.

What you are asking is for the links to be linkified in the review page itself before it is published to the bug?

dkl

Comment 2

[Christian :Biesinger (don't email me, ping me on IRC)]

(In reply to David Lawrence [:dkl] from comment #1)
> What you are asking is for the links to be linkified in the review page
> itself before it is published to the bug?

Yes.

Comment 3

[David Lawrence [:dkl]]

(In reply to Christian :Biesinger (don't email me, ping me on IRC) from comment #2)
> (In reply to David Lawrence [:dkl] from comment #1)
> > What you are asking is for the links to be linkified in the review page
> > itself before it is published to the bug?
> 
> Yes.

This will take some work as the previous comments shown in the Overview page are generated by JavasScript and the comments in show_bug.cgi are fed through a built in function that does the linkification. The JavaScript has no access to the same functions so it will have be done from scratch. 

dkl

Comment 4

[Kohei Yoshino [:kohei]]

Splinter is retiring.