237757 - Resolved bugs are no longer struck out on dependency trees

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, defect)

Description

[Dave Miller [:justdave]]

They are properly enclosed in class="bz_closed" however, it appears that the
stylesheet link at the top of the page is missing....

Comment 1

[Dave Miller [:justdave]]

This is a regression from bug 232397, apparently.  CCing the patch author and
committer from that bug.

Comment 2

[Christian Reis]

Attachment: kiko_v1: add link to edit_bug.css where necessary

This regressed in a number of places, actually.

The central problem here is that the edit_bug.css file isn't included anywhere
right now beyond bug/show.html.tmpl and the styles were added there. I think a
proper solution would be to use a central CSS file for bugzilla and store
certain "bugzilla-wide" styles there -- I guess bug 69654 would have gotten us
closer to this. 

My solution is crude, but it works. When we do fix this we'll have to go
a-greppin for the css links I added here though..

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 144112 [details] [diff] [review]
kiko_v1: add link to edit_bug.css where necessary

>Index: en/default/list/list.html.tmpl
  ...
>-[% style_urls = [ "css/buglist.css" ] %]
>+[% style_urls = [ "css/buglist.css" "css/edit_bug.css" ] %]

You don't need it in this file, since bug lists don't strike through resolved
bugs.

Overall, I guess this is OK, although I'd much prefer for you to create a
"global.css" and move these style rules there.

Comment 4

[Christian Reis]

(In reply to comment #3)
> (From update of attachment 144112 [details] [diff] [review])
> >Index: en/default/list/list.html.tmpl
>   ...
> >-[% style_urls = [ "css/buglist.css" ] %]
> >+[% style_urls = [ "css/buglist.css" "css/edit_bug.css" ] %]
> 
> You don't need it in this file, since bug lists don't strike through resolved
> bugs.

You missed the fact that list.html.tmpl is what processes edit-multiple.html.tmpl.

> Overall, I guess this is OK, although I'd much prefer for you to create a
> "global.css" and move these style rules there.

Do you endorse this officially? Can I add it as a default stylesheet to
header.html.tmpl?

Comment 5

[Myk Melez [:myk] [@mykmelez]]

> You missed the fact that list.html.tmpl is what processes edit-multiple.html.tmpl.

Ah, ok, never mind then.

> > Overall, I guess this is OK, although I'd much prefer for you to create a
> > "global.css" and move these style rules there.
> 
> Do you endorse this officially? Can I add it as a default stylesheet to
> header.html.tmpl?

Yes, go ahead and add it.  Please do so in a way which is compatible with bug
69654 and post a note in that bug after checking in so its assignee knows to
take your changes into account.

Comment 6

[Christian Reis]

Attachment: kiko_v2: do the right thing<tm>

Comment 7

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 144160 [details] [diff] [review]
kiko_v2: do the right thing<tm>

>Index: css/edit_bug.css
...
>-.bz_private { color: darkred ; background : #f3eeee ; }
>-.bz_disabled { color: #a0a0a0 ; }
>-
>-.bz_obsolete { text-decoration: line-through; }
>-.bz_inactive { text-decoration: line-through; }
>-.bz_closed { text-decoration: line-through; }

Let's contain the scope of this fix to just the changes in bug 232397, i.e.
don't move bz_private and bz_disabled over to global.css at this time.


>Index: css/global.css

This needs a license header (see buglist.css for an example).


>Index: template/en/default/global/header.html.tmpl
...
>       </style>
>     [% END %]
> 
>+    <link href="css/global.css" rel="stylesheet" type="text/css">

This should appear before the local style rules so that the local rules can
override it, or doesn't it matter?

Comment 8

[Christian Reis]

Attachment: kiko_v3: use MPL header, place link in the right place

It makes sense to have the global link come before the local styles -- I would
think the same for all linked style files, actually, but I'm not going to do
that here.

Comment 9

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 144165 [details] [diff] [review]
kiko_v3: use MPL header, place link in the right place

Looks good, works. r=myk

Comment 10

[Christian Reis]

Checked in (with no redness, hooray)