248379 - Convert Bugzilla footer to html4 and css (addon for bug 245924)

Status: RESOLVED FIXED

Description

[Svetlana Harisova]

In patch to bug 245924 we don't take into account that some users
will may want to style by css different group of links
in are different ways (e.g. actions-group will be green, 
edit-group will be yellow and saved-searches group will be
an other favorite color).

The our previous implementation of footer has made this impossible, 
because we assign ID not for all group=links+name_of_group but only for 
name_of_group:

<div class="group">
    <div id="link-edit">Edit:</div>
    <div class="links">
    ......
    </div>
</div>

The present CSS specification doesn't allow to access parent
properties from child rule.

We can assign ID for group=links+name_of_group, and when all
other users can design by CSS both whole group and
links or name_of_group with help of context selectors.

We have rewritten our previous patch from this point of view.
Now code looks like:

<div id="links-edit">
<div class="label">Edit:</div>
    <div class="links">
    ......
    </div>
</div>

Sorry for your inconvenience.

Comment 1

[Svetlana Harisova]

Created attachment 151552 [details] [diff] [review]
Provides an ability to style different group of links in diffrent ways in footer

Comment 2

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

Comment on attachment 151552 [details] [diff] [review]
Provides an ability to style different group of links in diffrent ways in footer

The change makes sense, but this patch causes the content of the useful links
box to get unnecessary extra top padding.  Strangely, you didn't change any
padding in this patch.	Perhaps the existing padding-top rule gets
misinterpreted for some reason?

Comment 3

[Svetlana Harisova]

Created attachment 152127 [details] [diff] [review]
Second attempt.

We have corrected code as far as could understand you.

If it is not difficult for you, next time specify browser and its version, in
which you observe one or another defect, please. It will help us very much.

If you have opportunity, the best variant is to give screenshot with defect.

Comment 4

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

Comment on attachment 152127 [details] [diff] [review]
Second attempt.

Sorry, I should have known you'd need to know the browser. :-/

I tested in Mozilla Firefox 0.9 on Linux.  The new patch still changes
formatting on the same browser, but less so, and it still looks good. r=myk

Comment 5

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

Minor low-risk update to previously checked in patch. a=myk and checking in for
contributor without CVS access:

Checking in css/global.css;
/cvsroot/mozilla/webtools/bugzilla/css/global.css,v  <--  global.css
new revision: 1.4; previous revision: 1.3
done
Checking in template/en/default/global/useful-links.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/global/useful-links.html.tmpl,v
 <--  useful-links.html.tmpl
new revision: 1.27; previous revision: 1.26
done