After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 168348 - [patch]theme-manager must not display "Hidden=true" Icons theme
[patch]theme-manager must not display "Hidden=true" Icons theme
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: [obsolete] theme-manager
git master
Other All
: High normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
: 160303 (view as bug list)
Depends on:
Blocks: 168977
 
 
Reported: 2005-02-24 06:29 UTC by shakti
Modified: 2005-05-22 16:15 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Patch fixes the problem (2.03 KB, patch)
2005-03-02 09:40 UTC, shakti
committed Details | Review

Description shakti 2005-02-24 06:29:38 UTC
Please describe the problem:
When a user selects 'Hicolor' in theme details the desktop icons and many other
icons disappear.

Steps to reproduce:
1. Open Desktop->Preferences->Theme
2. Click 'Theme Details' and select the 'Icons' tab
3. Select 'Hicolor' from the list.


Actual results:
The desktop icons disappear

Expected results:
The desktop icons should change to some (hicolor) icons.

Does this happen every time?
Yes.

Other information:
Comment 1 shakti 2005-02-24 06:38:17 UTC
Mauy be due to /opt/gnome2/share/icons/hicolor/48x48/filesystems is empty. Also,
36x36/filesystems, 64x64/filesystems etc are empty.
Comment 2 Baptiste Mille-Mathias 2005-02-24 06:42:36 UTC
Hello

AFAIK I think the hicolor theme should be displayed in the list of "Icons",
regarding the following lines in the files index.theme :

"...This is the default fallback theme used by implementations of the icon
theme specification...."

and then the theme set to be hidden :
Hidden=true

(I admit I've already seen this bug for few release, and I not opened a bug.
Blame on me)
Comment 3 Sebastien Bacher 2005-02-24 12:25:18 UTC
how is that a control-center bug ?
Comment 4 Baptiste Mille-Mathias 2005-02-24 16:53:22 UTC
Re,

I made a mistake in my previous comment; replace "should" by "shouldn't".

I rename this bug because the real problem is that the theme-manager shouldn't
display the icon theme Hicolor.

So this is a control-center bug :)
Comment 5 Christian Kirbach 2005-02-24 22:03:34 UTC
I guess Baptiste is correct. Theme manager is assigned to control center 
maintainers.
"theme-manager 	control-center-maint@bugzilla.gnome.org"

I can confirm that at least the desktop icons of quite some themes are
broken/incomplete (Amaranth, HiColor, Industrial...) in Gnome 2.9.91 .
*Maybe* some files just were not copied into their respective place as
some /share/icons/*/* directories are empty.

=> I think this really must be tackled before releasing 2.10 .

However I am not sure about the Hidden=true option and its meaning.
The only reference I found is

http://www.gnome.org/learn/admin-guide/latest/ch03s02.html
Comment 6 Christian Kirbach 2005-02-24 22:07:59 UTC
Baptiste please change the topic to something like
"Some icon themes are incomplete"

they are handled properly as the missing files are replaced
with a standard icon.

I am not sure if "Hidden=true" theme.index themes should be hidden
in the theme manager list.
Comment 7 Christian Kirbach 2005-02-24 22:39:06 UTC
according to #bugs share/icons/hicolor/  is a tree
and it doesn't have icons.

However "Hidden=true" in theme.index should indeed hide the
theme in the theme manager list.

thus please change topic to "Theme manager details icon list shows
themes tagged Hidden=true" or similar.
Comment 8 shakti 2005-02-25 04:05:57 UTC
The description of the key Hidden is:

"Whether to hide the theme in a theme selection user interface. This is used for
things such as fallback-themes that are not supposed to be visible to the user".

http://standards.freedesktop.org/icon-theme-spec/latest/ar01s04.html
Comment 9 Christian Kirbach 2005-02-25 08:27:04 UTC
Ok, theme should not be listed then.

The subdirs of Amaranth, HiColor, Industrial... themes are empty in
my garnome build. This is why a fallback happens to standard icon(s)
Should it be filed as a sperate themes bug?
Comment 10 Baptiste Mille-Mathias 2005-02-25 08:35:53 UTC
Christian,

Fill another bug concerning your problem
Comment 11 shakti 2005-03-02 09:40:56 UTC
Created attachment 38143 [details] [review]
Patch fixes the problem
Comment 12 shakti 2005-03-02 09:44:10 UTC
If there is a key named 'Hidden' and value is 'true', then don't add the Icon
Theme. The 'Icons' tab won't show it in the list.
Comment 13 Christian Kirbach 2005-03-02 10:17:39 UTC
Patch applies cleanly and works.

Raising priority to High to make maintainers aware of it.
Comment 14 Christian Kirbach 2005-03-02 10:30:19 UTC
*** Bug 160303 has been marked as a duplicate of this bug. ***
Comment 15 James Henstridge 2005-03-02 14:37:48 UTC
Add bug 168977 as a dependency of this one.  There are a bunch of a11y themes
that should probably be deprecated and hidden, which would clean up the dialog a
bit.
Comment 16 Sebastien Bacher 2005-05-22 16:15:37 UTC
patch commited, thanks