59515 - Add TITLE attribute to A HREF of bug # in description

Status: VERIFIED FIXED

(Bugzilla :: Bugzilla-General, enhancement, P3)

Description

[Jacob Steenhagen]

I like the way Asa has started adding the bug summary as a TITLE attribute of
bugs he links to at http://www.mozillazine.org/build_comments/  This makes it
very easy to simply hold the mouse over the link and get a clue about what a bug
is without having to click on it and load another page.  This could also be
useful for the dependency tree and even Bonsai (although I realize that's a
completely different script).

Comment 1

[Jacob Steenhagen]

Attachment: Patch to add title= to dependencies

Comment 2

[Jacob Steenhagen]

OK, I'm no SQL expert (in fact, I've never used) but looking at the code I think
I figured out how to add this feature to the dependancies.  Also in looking at
the code, it appears that in order to add it the the desctiption entries, it
would require the addition of an SQL call for each bug link.  I don't know if
this is something that would be desired...

Comment 3

[Jacob Steenhagen]

Attachment: Add title= to description (quoteUrls)

Comment 4

[Jacob Steenhagen]

OK, I went ahead and made a diff file for adding this functionality to the
description within the quoteUrls subroutine.  If it's decided that this is a
good idea (worth the extra SQL call) then I'd recommend double (and maybe even
triple) checking my syntax for the SQL call.

Comment 5

[Dan Mosedale (:dmosedale, :dmose)]

I really like this idea.

Comment 6

[Jacob Steenhagen]

OK, I finally got a working hackable bugzilla installation (of all places, it's
installed on my Laptop).

Anyway, I've tested both these patches and they work as expected.

Comment 7

[Dan Mosedale (:dmosedale, :dmose)]

I just checked these patches in; thanks Jake.  Next time we update
bugzilla.mozilla.org to the CVS tip (and once tooltips get un-busted in the
Mozilla build), the world will see the fruits of your labor.

Comment 8

[Jesse Ruderman]

On landfill, I see the status but not the resolution in the tooltip.  I think 
it would be more useful to list the resolution, or to list both the status and 
the resolution.

It seems like this code runs every time I grab a bug report with show_bug.cgi. 
Doesn't the extra database queries cause a performance hit?  It might make 
sense to make the comments static, but fix bug 12286 (keeping track of which 
bugs link to which other bugs) and reparse a bug when linked-to bugs have their 
status/resolution changed.

(I could be completely misinterpreting the situation, because I'm not familiar 
with how bugzilla works.)

Comment 9

[Jacob Steenhagen]

I think what should be done is a "GetBugTitle" function added to globals.pl.  
This would give the ability to cache a title tag after running the DB query so 
at least the SQL server is only queried once per bug # per page load. (For bugs 
where the same bug # is referenced multiple time [not uncommon] this would 
really help cut down on the number of queries.  The biggest problem with this 
is the dependancy tree.  The way it's subroutine runs (which is a direct 
resault of the way the tree is stored) would make calling a subroutine such as 
this difficult.  I'll see what I can come up with... maybe it will be workable.

The idea of including the resolution is a good one... I'll also work on that 
ASAP.

Comment 10

[Jacob Steenhagen]

Jesse, this resolution in the tooltip problem is addressed in a patch attached 
to bug 30694.

Comment 11

[Dave Miller [:justdave]]

In search of accurate queries....  (sorry for the spam)

Comment 12

[Jacob Steenhagen]

V.... Even the extra bit about Resolution has been fixed.

Comment 13

[Dave Miller [:justdave]]

Moving closed bugs to Bugzilla product