Commit aec17c9f authored by Christian Legnitto's avatar Christian Legnitto Committed by Max Kanat-Alexander

Bug 585490: Instead of hardcoding <i> for UNCONFIRMED bug links, use

a CSS class. r=mkanat, a=mkanat
parent f88ae24f
...@@ -329,17 +329,16 @@ sub get_bug_link { ...@@ -329,17 +329,16 @@ sub get_bug_link {
# Initialize these variables to be "" so that we don't get warnings # Initialize these variables to be "" so that we don't get warnings
# if we don't change them below (which is highly likely). # if we don't change them below (which is highly likely).
my ($pre, $title, $post) = ("", "", ""); my ($pre, $title, $post) = ("", "", "");
my @css_classes = ("bz_bug_link");
$title = get_text('get_status', { status => $bug->bug_status }); $title = get_text('get_status', { status => $bug->bug_status });
if ($bug->bug_status eq 'UNCONFIRMED') {
$pre = "<i>"; push @css_classes, "bz_status_" . css_class_quote($bug->bug_status);
$post = "</i>";
}
if ($bug->resolution) { if ($bug->resolution) {
$pre .= '<span class="bz_closed">'; push @css_classes, "bz_closed";
$title .= ' ' . get_text('get_resolution', $title .= ' ' . get_text('get_resolution',
{ resolution => $bug->resolution }); { resolution => $bug->resolution });
$post .= '</span>';
} }
if (Bugzilla->user->can_see_bug($bug)) { if (Bugzilla->user->can_see_bug($bug)) {
$title .= " - " . $bug->short_desc; $title .= " - " . $bug->short_desc;
...@@ -354,6 +353,10 @@ sub get_bug_link { ...@@ -354,6 +353,10 @@ sub get_bug_link {
if (defined $options->{comment_num}) { if (defined $options->{comment_num}) {
$linkval .= "#c" . $options->{comment_num}; $linkval .= "#c" . $options->{comment_num};
} }
$pre = '<span class="' . join(" ", @css_classes) . '">';
$post = '</span>';
return qq{$pre<a href="$linkval" title="$title">$link_text</a>$post}; return qq{$pre<a href="$linkval" title="$title">$link_text</a>$post};
} }
......
...@@ -251,6 +251,18 @@ div#docslinks { ...@@ -251,6 +251,18 @@ div#docslinks {
margin: 0; margin: 0;
} }
/**************************/
/* Bug links and statuses */
/**************************/
.bz_bug_link {
/* Catch-all if you want common styles for all bug links */
}
.bz_bug_link .bz_status_UNCONFIRMED {
font-style: italic;
}
.bz_obsolete { .bz_obsolete {
text-decoration: line-through; text-decoration: line-through;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment