missing about icon in win32

Bug #371724 reported by theAdib
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Unassigned
Nominated for 0.47.x by theAdib
Nominated for Old by Lorenz Aebi

Bug Description

The Icon for the about menu entry on win32 is missing see the sceenshot

Revision history for this message
theAdib (theadib) wrote :
tags: added: ui win32
Revision history for this message
nightrow (jb-benoit) wrote :

Using inkscape 0.46 on WinXP, the icon is displayed at it should.

Revision history for this message
theAdib (theadib) wrote :

sorry, I mean the current development version. ie. version from date 20090504.

Revision history for this message
jazzynico (jazzynico) wrote :

Confirmed on Vista, Inkscape build 21271.

Changed in inkscape:
status: New → Confirmed
Revision history for this message
Alvin Penner (apenner) wrote :

- confirmed on Windows XP as well in build 21271
- this icon was visible previously in build 21210

Revision history for this message
LucaDC (lucadc) wrote :

Probably you already know, but when you open the Help menu (where the icon is missing) the console says:

(inkscape.exe:2672): GLib-CRITICAL **: g_convert: assertion `str != NULL' failed
(inkscape.exe:2672): GLib-GIO-CRITICAL **: g_loadable_icon_load: assertion `G_IS_LOADABLE_ICON (icon)' failed
(inkscape.exe:2672): GLib-CRITICAL **: g_convert: assertion `str != NULL' failed
(inkscape.exe:2672): GLib-GIO-CRITICAL **: g_loadable_icon_load: assertion `G_IS_LOADABLE_ICON (icon)' failed

Win XP SP3, SVN 21297.

Revision history for this message
Lorenz Aebi (inkscape-xca) wrote :

if you copy the icon inkscape.png or inkscape.svg from the folder /share/icons/tango/scalable to the folder /share/icons/tango/scalable/places if you do this for every possibility then the icons are shown correct.

I don't know how to solve this problem. There is a problem in the paths definition (Makefile.am).

Revision history for this message
Lorenz Aebi (inkscape-xca) wrote :

the index.theme file is wrong.

Revision history for this message
Lorenz Aebi (inkscape-xca) wrote :

next try:

remove the inkscape.svg file in the inkscape\share\icons folder. Then it works on my pc.

Revision history for this message
Alvin Penner (apenner) wrote :

thanks, removing inkscape.svg works for me as well on Windows XP, build 21345.

Now, if only we knew why...

Revision history for this message
Lorenz Aebi (inkscape-xca) wrote :

what i found is that the problem is in the widgets\icon.cpp file. i did a patch, but i really don't know if this is correct like this. at least it compiles, inkscape starts up and the icon is shown...

Revision history for this message
Lorenz Aebi (inkscape-xca) wrote :

i did some research but i don't know why we need the "static std::set<Glib::ustring> internalNames" variable. Can anybody explain me?

Because with out this variable the program is still working. See patch

jazzynico (jazzynico)
Changed in inkscape:
importance: Undecided → Low
milestone: none → 0.47
tags: added: regression
Revision history for this message
su_v (suv-lp) wrote :

Happens on MacOS X 10.5.7 with Inkscape 0.47pre0 r21549, built Jun 13 2009 as well.

My guess is, it's related to the icon name/id (mismatch with hardcoded name in src?)
  id = "inkscape"
  inkscape:label = "#inkscape_options"
in icons.svg. tweenk or joncruz ought to have a look at this!

Don't remove that file - AFAIK hicolor (tango) icons will not be included in 0.47, and all inkscape 0.47 icons are in 'icons/icons.svg', as in earlier releases.

tags: added: osx
Revision history for this message
su_v (suv-lp) wrote : Re: [Bug 371724] Re: missing about icon in win32

Argh, I overlooked the filename 'inkscake.svg' in previous comments (7,9,10)
Renaming 'icons/inkscape.svg' forces Inkscape 0.47pre0 to load the about-icon from 'icons/icons.svg', which renders fine.

Builds with certain build/platform configurations (gtk/gio, librsvg?) seem to fail to load the scalable inkscape icon (orig size: 128x128 px, rendered size in menu 16x16 px). At least that's the main difference between r21525 (from modevia, JiHO) and 0.47pre0 r21549 (from sourceforge, mjwybrow) builds for OS X - I could not find SVN commits between 21525 and 21549 relevant to the widget/icon mechanism.

I compared the icon debug output (as descibed in http://wiki.inkscape.org/wiki/index.php/Icons#Debug_Output) between

  Inkscape 0.46+devel r21525 (about-icon is displayed)
  Inkscape 0.47pre0 r21549 (about-icon is missing)

Both are locally configured to use 'icons/icons.svg' and not the hicolor named icon theme.
output regarding the 'about-icon' (for both the same except the 0x value):

** Message: gtk_image_new_from_icon_name( 'inkscape', 1 ) = 0x45a2d68
** Message: Type is 7 ok
<…>
** Message: prerender_icon [inkscape] 16 NOT!!!!!!

The only relevant differences are (as in comment #6 already mentioned) messages from Inkscape 0.47pre0, they occur when opening the help menu:

(inkscape-bin:3341): GLib-GIO-CRITICAL **: g_loadable_icon_load: assertion `G_IS_LOADABLE_ICON (icon)' failed
(inkscape-bin:3341): GLib-GIO-CRITICAL **: g_loadable_icon_load: assertion `G_IS_LOADABLE_ICON (icon)' failed

Revision history for this message
Alvin Penner (apenner) wrote :

just writing to reaffirm that this bug is still present in the Windows build (7z file) for 0.47pre0 , July 2.

This is a very visible bug, for anyone trying to get help on Inkscape...

Revision history for this message
Alvin Penner (apenner) wrote :

just writing to reaffirm that this bug is still present in the Windows build (7z file) for 0.47pre1 , July 3.

Revision history for this message
su_v (suv-lp) wrote :

confirmed with Inkscape 0.47pre1 r21720, built Jul 20 2009 on OS X 10.5.7

Revision history for this message
su_v (suv-lp) wrote :

just built Inkscape r21924 on OS X 10.5.7:
librsvg (i.e. on osx: port install librsvg) installs the 'svg_loader'
module and adds 'svg' back to the list in 'gdk-pixbuf.loaders'. Both are
included into the application bundle (packaging process after
re-building). It seems to fix this bug: the about-icon is correctly
rendered, no more console warnings when opening the help menu.

test:
1) install librsvg via MacPorts

| LeWitt:~ suv$ port installed | grep librsvg
| librsvg @2.26.0_0 (active)

2) rebuild Inkscape, create application bundle
   run Inkscape.app: the about-icon is correctly rendered
3) quit Inkscape
   in 'Contents/Resources/etc/gtk-2.0/gdk-pixbuf.loaders'
   comment out the lines:

| "${CWD}/lib/gtk-2.0/2.10.0/loaders/svg_loader.so"
| "svg" 2 "gtk20" "Scalable Vector Graphics" "LGPL"
| "image/svg+xml" "image/svg" "image/svg-xml" "image/vnd.adobe.svg+xml"
| "text/xml-svg" "image/svg+xml-compressed" ""
| "svg" "svgz" "svg.gz" ""
| " <svg" "* " 100
| " <!DOCTYPE svg" "* " 100

4) restart Inkscape.app

result:
the about-icon is again missing.

Unfortunately I don't know if this relates in any way to the issue
(missing about-icon) with Windows builds.

Revision history for this message
Krzysztof Kosinski (tweenk) wrote :

It looks like the GTK function gtk_icon_theme_has_icon is slightly broken, as it returns true if only an SVG icon exists for the given name but the SVG loader is not installed. The possible solutions are:

a) bundle librsvg on Windows and Mac OS
b) change the call to gtk_icon_theme_has_icon in widgets/icon.h to gtk_icon_theme_lookup_icon

I have to finish my GSoC project so I can't test those solutions properly.

Revision history for this message
su_v (suv-lp) wrote :

SVN rev. 21987 commits a workaround (renaming 'inkscape' to 'inkscape-logo' in icons.svg and changing the icon remapping) to fix the issue (mainly for win32) - but the underlaying cause of the missling librsvg / SVG loader (as confirmed by Krzysztof) still needs to be addressed and fixed in my opinion.

Revision history for this message
Alvin Penner (apenner) wrote :

thanks, ~suv and Krzysztof,
that patch is working well in the Windows version of 0.47pre2

Revision history for this message
su_v (suv-lp) wrote :

@Alvin - the workaround was committed by Chris Morgan. The underlying issue is not yet addressed...

ScislaC (scislac)
Changed in inkscape:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.