1078096 - Include the target milestone in the tooltip when hovering a bug ID

Status: RESOLVED DUPLICATE of bug 1146763

(Bugzilla :: Bugzilla-General, enhancement)

Description

[Frédéric Buclin]

When you hover a bug ID e.g. in a comment, the info you get in the tooltip is:

  STATUS RESOLUTION - Bug Summary

Most of the time, when the resolution is FIXED, I would like to also know *when* the bug was fixed, i.e. I want to know its target milestone. I suggest to change the tooltip to:

  STATUS RESOLUTION (MILESTONE) - Bug Summary

The milestone should only be displayed if the bug is accessible, as we already do for the bug summary.

Comment 1

[Sunil Joshi [:sjoshi]]

Attachment: Patchv1.diff

Displays Target Milestone if "usetargetmilestone" is set to ON in the tooltip on mouse hover to a BugId in comments section.

Comment 2

[Frédéric Buclin]

Comment on attachment 8504240 [details] [diff] [review]
Patchv1.diff

> [% IF user.can_see_bug(bug) %]
>+    [% IF Param("usetargetmilestone") %]
>+      [% link_title = link_title _ ' (' _ display_value('target_milestone', bug.target_milestone) _ ')' %]
>+    [% END %]

Fix the indentation, please. Templates use 2 whitespaces, not 4.

Comment 3

[Sunil Joshi [:sjoshi]]

Attachment: Bug#1078096.diff

Fixed the indentation problem.

Comment 4

[Gervase Markham [:gerv]]

Comment on attachment 8505584 [details] [diff] [review]
Bug#1078096.diff

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

::: template/en/default/bug/link.html.tmpl
@@ +31,5 @@
>  
>  [% IF user.can_see_bug(bug) %]
> +  [% IF Param("usetargetmilestone") %]
> +    [% link_title = link_title _ ' (' _ display_value('target_milestone', bug.target_milestone) _ ')' %]
> +  [% END %]

We shouldn't add this if the target milestone is "---". 

(---)
 
is not a helpful and clear addition to this tooltip.

Gerv

Comment 5

[Frédéric Buclin]

(In reply to Gervase Markham [:gerv] from comment #4)
> We shouldn't add this if the target milestone is "---". 

Why? Who said this is not useful? It's very useful to easily track if a bug had no milestone set. Same when a bug is not FIXED but has a milestone set, etc... depending on the policy used for each installation.

Comment 6

[Frédéric Buclin]

Well, one could easily understand that no milestone visible means ---. OK, let's say I can live with that.