244940 - 'loading.gif' is found in many different places: standardize it in one location!

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect)

Description

[Alfred Kayser]

In the 'Classic' and 'Modern' themes, the image loading.gif is all over the place:

[code]
/themes/classic/communicator/bookmarks/bookmarks.css, line 67 --
list-style-image: url("chrome://global/skin/loading.gif") !important;
/themes/classic/communicator/directory/directory.css, line 65 --
list-style-image: url("chrome://communicator/skin/icons/loading.gif");
/themes/classic/communicator/related/related.css, line 76 -- list-style-image:
url("chrome://global/skin/loading.gif") ! important ;
/themes/classic/communicator/search/search.css, line 45 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/themes/classic/communicator/sidebar/sidebar.css, line 111 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/themes/classic/global/mac/browser.css, line 36 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/themes/classic/global/win/browser.css, line 32 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/themes/modern/communicator/bookmarks/bookmarks.css, line 69 --
list-style-image: url("chrome://global/skin/loading.gif") !important;
/themes/modern/communicator/directory/directory.css, line 65 --
list-style-image: url("chrome://communicator/skin/icons/loading.gif");
/themes/modern/communicator/related/related.css, line 69 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif") !important ;
/themes/modern/communicator/search/search.css, line 45 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/themes/modern/communicator/sidebar/sidebar.css, line 113 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/themes/modern/global/browser.css, line 62 -- list-style-image:
url("chrome://communicator/skin/icons/loading.gif");
/toolkit/components/help/skin/help.css, line 130 -- list-style-image:
url("chrome://global/skin/icons/Loading.gif");
/toolkit/skin/win/browser.css, line 32 -- list-style-image:
url("chrome://global/skin/icons/Loading.gif");
/browser/base/skin/browser.css, line 713 -- list-style-image:
url("chrome://global/skin/icons/Loading.gif");
/browser/base/skin/browser.css, line 746 -- list-style-image:
url(chrome://global/skin/icons/Loading.gif);[/code]

Luckely the only references are in the in '/skin' part, so all changeable by the
themer.

Can we move 'loading.gif" to just '/global/skin/' (or /global/skin/icons/ if
really needed)?

If they are semantically different than use also a really distinguishing name...

Comment 1

[[e-mail]]

Well, toolkit doesn't have communicator/ so it needs to stuff everything into
global/ and so the only issue is that bookmarks and related weren't fixed when
hewitt moved loading.gif from global into communicator.

Or alternatively we could just move loading.gif back into global...

Comment 2

[Alfred Kayser]

That's indeed what should happen as a lot of components using loading.gif,
so it is pretty general, so /global/skin is a good place for it.

Comment 3

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) <bookmarks.css> [Checkin: Comment 6]

I don't know what the situation was in 2004,
but fixing this missed reference seems enough now.

Yet, tell me if this file should actually be moved back to "global"...

Comment 4

[Serge Gautherie (:sgautherie)]

The image file lives in
http://mxr.mozilla.org/comm-central/find?text=&kind=text&string=%2Floading.gif

Comment 5

[Stefan [:stefanh]]

That icon will look odd for mac classic, but I guess I can take care of that in bug 460699.

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 372279 [details] [diff] [review]
(Av1) <bookmarks.css>
[Checkin: Comment 6]


http://hg.mozilla.org/comm-central/rev/2dd61877111c