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 470404 - Usage of non-tango yelp-icon-big
Usage of non-tango yelp-icon-big
Status: RESOLVED FIXED
Product: yelp
Classification: Applications
Component: General
git master
Other Linux
: Normal normal
: ---
Assigned To: Yelp maintainers
Yelp maintainers
Depends on:
Blocks:
 
 
Reported: 2007-08-26 11:29 UTC by Michael Monreal
Modified: 2007-09-05 20:03 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
New Icon (29.93 KB, image/png)
2007-09-04 18:50 UTC, Don Scorgie
Details

Description Michael Monreal 2007-08-26 11:29:50 UTC
Currently Yelp ships a 192x192 bitmap icon named yelp-icon-big, which is not in tango style.

I tried to fix this by using the gnome-help icon, but didn't really understand how the icon is being used (how it gets from the theme to the HTML page)

We have the new tangoish Yelp icon available as an SVG in gnome-icon-theme, but I guess Yelp needs a bitmap here?
Comment 1 Don Scorgie 2007-09-03 19:20:23 UTC
The icon is used on the table of contents of yelp (when you first start it up), in the left column of categories, as a background image.  That's the only place it's used.  It's found in src/yelp-toc.c, around line 779 using gtk_icon_theme_lookup_icon at a size of 192x192.

What's the best way to handle this?  I tried using this directly, but something in gecko borks on it / the way I'm using it (resulting in no image) :(  We could do a direct replacement of the file if that's acceptable.  Or putting a 192x192 image in gnome-icon-theme?  I'm inclined to just do a copy the new icon into the yelp package, unless there's an objection.
Comment 2 Michael Monreal 2007-09-03 19:45:22 UTC
I guess the best way would be to render a big bitmap from the SVG and put it into the yelp package... and the icon should probably not be installed in hicolor, but a private icon directory, see http://live.gnome.org/ThemableAppSpecificIcons
Comment 3 Don Scorgie 2007-09-04 18:50:40 UTC
Created attachment 94949 [details]
New Icon

This is the new icon.  I've requested a UI break to put this into the yelp package for 2.20.  With this, I'll make the relevent changes and install in the correct place.
Comment 4 Don Scorgie 2007-09-05 18:10:54 UTC
Icon changed in SVN.  I've left the icon as public for now as it is the same icon as the "app" icon.  If people want to use the 192x192 version, it's up to them.  If you disagree strongly, please leave an explanation and I'll reconsider.

For now though, marking as Fixed.

2007-09-05  Don Scorgie  <dscorgie@svn.gnome.org>

        * data/icons/yelp-icon-big.png:
        New tango-ified version
Comment 5 Michael Monreal 2007-09-05 20:03:59 UTC
Is there a need to call the icon "yelp-icon-big" instead of just "yelp"?