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 327884 - gmenu-simple-editor icon sizes not equal
gmenu-simple-editor icon sizes not equal
Status: RESOLVED WONTFIX
Product: gnome-menus
Classification: Core
Component: general
git master
Other All
: Normal normal
: ---
Assigned To: gnome-menus dummy account
gnome-menus dummy account
Depends on:
Blocks:
 
 
Reported: 2006-01-20 16:39 UTC by Frank Arnold
Modified: 2006-05-04 19:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to scale the icon if needed (904 bytes, patch)
2006-01-20 16:40 UTC, Frank Arnold
none Details | Review

Description Frank Arnold 2006-01-20 16:39:40 UTC
Used it the first time.

For some applications there are no 24x24 icons available, like Dia for instance. gtk_icon_theme_load_icon () doesn't scale it to the specified size if it's near enough. This looks ugly.
Comment 1 Frank Arnold 2006-01-20 16:40:59 UTC
Created attachment 57738 [details] [review]
Patch to scale the icon if needed
Comment 2 Matthias Clasen 2006-03-24 15:38:01 UTC
IMO, this is just working around problems in the icon theme. 
If 24x24 icons are required, the app should install them, or
better, install a scalable version. Then there is no way to 
hack around the problem like this. 

Removing the same hack from the panel would probably help a lot
with fixing this problem at the right spot...
Comment 3 Frank Arnold 2006-03-24 21:16:03 UTC
How about dropping the whole thing silently? It never showed up in the menus by now, and the vast majority of users who wants to modify the menu seems to use alacarte or propably gnome-menu-editor.

For my usecase it didn't work either. I'm a one panel user. I wanted to get rid of the pirut entry in the combined panel menu (the one with an icon only), because it is overly cluttered. I can turn this entry off, but then I'm left with two separators which I can't remove.

IMO this is a dead born child...
As you can see, I wouldn't care about a CLOSED/WONTFIX.

Regarding the panel hack, if you need more bug spam just patch it for Fedora ;)
Comment 4 Matthias Clasen 2006-03-24 21:22:11 UTC
What do you mean it never showed up ? 
Right click on the menubar, and its right there.

> Regarding the panel hack, if you need more bug spam just patch it for Fedora ;)

Nope, thats not how it works. Upstream has to clean up its act too. 
Cf the fatal-warnings thing which turned out to help cleaning up a lot of
warnings, even though we turned it off in rawhide.
Comment 5 Frank Arnold 2006-03-24 22:27:58 UTC
(In reply to comment #4)
> What do you mean it never showed up ? 
> Right click on the menubar, and its right there.

Seems like some users never discovered this, including me...

> Nope, thats not how it works. Upstream has to clean up its act too. 

Hey, I was just kidding :)
Comment 6 Mark McLoughlin 2006-05-04 19:09:44 UTC
Agree with mclasen, we should fix the apps

(Also, if we were going to scale the icon we'd want to retain it's aspect ratio)