860297 - "see also" links shown by inline history which link to the current bugzilla installation should show tooltips

Status: RESOLVED FIXED

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

Description

[:glob ✱]

"see also" links shown by inline history which link to the current bugzilla installation should show tooltips

Comment 1

[Thomas D. (:thomas8)]

This would be a great thing to polish. It's quite annoying to see that link right in front of you in the comment with no way of knowing the summary of the bug it is pointing to (and status etc.), short of either going to the top of the bug or following the link.

As these bug link tooltip exists in several other places, it should be a no-brainer copy-paste to fix this for someone familiar with the code. Tia.

You could even consider shortening the link text to just "Bug xxxxxx" because I'm not sure there's a particular value of *seeing* the full URL for current bugzilla installation whilst it will still be available via status bar on hover or Copy Link Location from link's context menu.

Comment 2

[:glob ✱]

for anyone willing to pick up this bug, here's what's required:

- move the splitting of see_also from the template to extension's perl code
- split each see_also change on /, /
  - set $change->{value} to store a hashref of arrays instead of a delimited string
  - foreach url, detect if it's a reference to the current installation
- update add_change in the template to iterate each see_also arrayref
  - if the url is identified as local, render with bug_link filter
  - otherwise render as href

Comment 3

[:glob ✱]

Attachment: 860297_1.patch

this was annoying me today, so here's the fix.

Comment 5

[David Lawrence [:dkl]]

Comment on attachment 8525087 [details] [diff] [review]
860297_1.patch

Review of attachment 8525087 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. r=dkl

Comment 6

[:glob ✱]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   4ccf1f4..a670bb1  master -> master