[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Menu icon policy (Was: Bug#142818: xteddy: Icons missing from menu entries)



Tille, Andreas wrote:

> Thanks for the bug report.

Hi Andreas,

Sorry for the delay in replying. But I didn't receive your response directly 
(by default the BTS doesn't send followup messages to the bug submitter) .

> I just want to know which window manager you are using.  I did not found
> any policy statement about default icon locations.  The documentation
> of menu states:

I'm running KDE. icewm also appears to need the full path specified.  fvwm 
(and variants) seem to pick up from the default location.

>      ...
>      please put all icon files in the directory
>      `/usr/X11R6/include/X11/{bitmaps,pixmaps}'

It only states that that is where you should place them, it doesn't state 
anywhere that this is the default location for window managers to find them.
In fact it the menu documentation in it's only examples for icon specifies 
the full path every time. But you are correct it isn't clear:

!            icon=/usr/share/emacs/20.3/etc/emacs.xbm
...
!	   X11 Apps menu/apps /usr/X11R6/include/X11/pixmap/icon.xpm "Editors"
...
!                     icon=path-to-pixmap.xpm\

> ~> grep "icon" /usr/lib/menu/* 2>/dev/null | grep -v
> /usr/X11R6/include/X11 | wc -l
>      47
> ~> grep "icon" /usr/lib/menu/* 2>/dev/null | grep /usr/X11R6/include/X11 |
> wc -l
>      19

I think your maths is a little out. Have a look at exactly which files don't 
contain /usr/X11R6/include/X11.  On my system the -v case returns the 
following:

msp@htpc:~$ grep "icon" /usr/lib/menu/* 2>/dev/null | grep -v 
/usr/X11R6/include/X11
/usr/lib/menu/amor:     icon="/usr/share/pixmaps/amor.xpm"\
/usr/lib/menu/aumix:    icon=/usr/share/pixmaps/icon_aumix.xpm\
/usr/lib/menu/avifile-player:   icon="/usr/share/avifile0.7/test.xpm" \
/usr/lib/menu/avifile-samples:  icon="/usr/share/avifile0.6/test.xpm" \
/usr/lib/menu/avifile-samples:  icon="/usr/share/avifile0.6/test.xpm" \
/usr/lib/menu/csmash:  icon="/usr/share/games/csmash/images/csmash-icon.xpm"
/usr/lib/menu/doc-linux-html: description="How to use escape sequences to 
dynamically change window and icon titles of an xte
rm." \
/usr/lib/menu/etherape:  icon="/usr/share/pixmaps/etherape.png"
/usr/lib/menu/everybuddy:  command="/usr/bin/everybuddy" 
icon="/usr/share/pixmaps/ebicon.xpm"
/usr/lib/menu/gltron:  icon="/usr/share/pixmaps/gltron.xpm"
/usr/lib/menu/heroes-sdl:icon=/usr/include/X11/pixmaps/heroes.xpm
/usr/lib/menu/heroes-sdl:icon=/usr/include/X11/pixmaps/heroes.xpm
/usr/lib/menu/heroes-sdl.orig:icon=/usr/include/X11/pixmaps/heroes.xpm
/usr/lib/menu/heroes-sdl.orig:icon=/usr/include/X11/pixmaps/heroes.xpm
/usr/lib/menu/j2re1.3:        icon="/usr/share/pixmaps/java-preferences.png"
/usr/lib/menu/j2re1.3:        icon="/usr/share/pixmaps/java-properties.png"
/usr/lib/menu/kchart:   icon="/usr/share/pixmaps/kchart.xpm"\
/usr/lib/menu/kcpuload:  icon="/usr/share/pixmaps/kcpuload.xpm"
/usr/lib/menu/kformula: icon="/usr/share/pixmaps/kformula.xpm"\
/usr/lib/menu/kgeo:  icon="/usr/share/pixmaps/kgeo.xpm"
/usr/lib/menu/kiconedit:?package(kiconedit):\
/usr/lib/menu/kiconedit:        command=/usr/bin/kiconedit
/usr/lib/menu/kivio:    icon="/usr/share/pixmaps/kivio.xpm"\
/usr/lib/menu/kmoon:    icon="/usr/share/pixmaps/kmoon.xpm"\
/usr/lib/menu/knetload:  icon="/usr/share/pixmaps/knetload.xpm"
/usr/lib/menu/kodo:     icon="/usr/share/pixmaps/kodo.xpm"\
/usr/lib/menu/komba2:   icon="/usr/share/icons/locolor/32x32/apps/komba2.xpm"
/usr/lib/menu/kontour:  icon="/usr/share/pixmaps/kontour.xpm"\
/usr/lib/menu/kpresenter:       icon="/usr/share/pixmaps/kpresenter.xpm"\
/usr/lib/menu/ksensors: icon="/usr/share/pixmaps/ksensors.xpm"
/usr/lib/menu/kspread:  icon="/usr/share/pixmaps/kspread.xpm"\
/usr/lib/menu/kugar:    icon="/usr/share/pixmaps/kugar.xpm"\
/usr/lib/menu/kword:    icon="/usr/share/pixmaps/kword.xpm"\
/usr/lib/menu/kworldclock:      icon="/usr/share/pixmaps/kworldclock.xpm"\
/usr/lib/menu/pan:  title="Pan" command="/usr/bin/pan" hints="News Reader" 
icon=/usr/share/pixmaps/pan.png
/usr/lib/menu/powermanga:  icon="/usr/share/games/powermanga/powermanga.xpm"
/usr/lib/menu/realplayer:       icon="/usr/share/pixmaps/realplayer-icon.xpm" 
\
/usr/lib/menu/showimg:  icon="/usr/share/icons/hicolor/32x32/apps/ShowImg.xpm"
/usr/lib/menu/tipptrainer:  title="Tipptrainer" icon="tipptrainer.xpm" 
command="/usr/bin/tipptrainer"
/usr/lib/menu/trophy:   needs="x11" icon="trophy.xpm" section="Games/Arcade" \
/usr/lib/menu/tuxkart:  title="Tuxkart" command="/usr/games/tuxkart" 
icon="tuxkart.xpm"
/usr/lib/menu/tuxracer:  title="TuxRacer" command="/usr/games/tuxracer" 
icon="tuxracer.xpm"
/usr/lib/menu/vlc:               icon="/usr/share/videolan/vlc.xpm" 
hints="Video"
/usr/lib/menu/vlc-gnome:                    
icon="/usr/share/videolan/gnome-vlc.xpm" hints="Video"
/usr/lib/menu/vlc-qt:                 icon="/usr/share/videolan/qvlc.xpm" 
hints="Video"
/usr/lib/menu/xteddy:  title="Teddy" icon="teddy.xpm" 
command="/usr/bin/X11/teddy"
/usr/lib/menu/xteddy:  title="XAlex" icon="xalex.xpm" 
command="/usr/bin/X11/xalex"
/usr/lib/menu/xteddy:  title="XBobo" icon="xbobo.xpm" 
command="/usr/bin/X11/xbobo"
/usr/lib/menu/xteddy:  title="XBrummi" icon="xbrummi.xpm" 
command="/usr/bin/X11/xbrummi"
/usr/lib/menu/xteddy:  title="XDuck" icon="xduck.xpm" 
command="/usr/bin/X11/xduck"
/usr/lib/menu/xteddy:  title="XKlitze" icon="xklitze.xpm" 
command="/usr/bin/X11/xklitze"
/usr/lib/menu/xteddy:  title="XNamu" icon="xnamu.xpm" 
command="/usr/bin/X11/xnamu"
/usr/lib/menu/xteddy:  title="XOrca" icon="xorca.xpm" 
command="/usr/bin/X11/xorca"
/usr/lib/menu/xteddy:  title="XPenguin" icon="xpenguin.xpm" 
command="/usr/bin/X11/xpenguin"
/usr/lib/menu/xteddy:  title="XRuessel" icon="xruessel.xpm" 
command="/usr/bin/X11/xruessel"
/usr/lib/menu/xteddy:  title="XTeddy" icon="xteddy.xpm" 
command="/usr/bin/X11/xteddy"
/usr/lib/menu/xteddy:  title="XTuxxy" icon="xtuxxy.xpm" 
command="/usr/bin/X11/xtuxxy"

As you can see most of those applications do specify a full path (but arn't 
located in the default location)

> Would you really like to say me that you want to browse /usr/lib/menu for
> icons
> with no full path specification and file bug reports?  In my opinion the
> right way to go would to write a policy first. 

I have actually filed bug reports, because the icons don't display correctly 
under KDE as they are supposed to. (I think there was less than 6 packages)

There wern't very many packages and they are easy to spot as the icon was 
missing from my menu... :-)

I guess my patch is just one way to solve the problem. Your alternatives are 
another way to solve the problem.  Either way the bug should remain until the 
icons are displayed again. (transferred to an alternative package if 
necessary)

Mark


> There could be
> posibilities:
> 
>   1. (Complienace with /usr/share/doc/menu/menu.txt.gz = see above)
>      Define a fixed path to icons and every window manager (menu using
>      application has to look there in case it does not find the specified
>      icon.
> 
>   or
> 
>   2. All applications have to specify the complete path name to their
>   icons.
> 
> I would prefer the first.
> 
> Any comments?
> 
> Kind regards
> 
>          Andreas.
> 
> PS: For those on Debian-Devel who do not like browsing BTS:
> 
> On Sun, 14 Apr 2002, Mark Purcell wrote:
> 
>> Package: xteddy
>> Version: 2.0.1-6
>> Severity: normal
>> Tags: patch
>>
>> Andreas,
>>
>> It appears that you need to specify the full icon path for menu to
>> function correctly.
>>
>> Mark
>>
>>
>> --- /usr/lib/menu/xteddy	Thu Jan 17 01:11:09 2002
>> +++ /etc/menu/xteddy	Sun Apr 14 19:56:13 2002
>> @@ -1,24 +1,24 @@
>>  ?package(xteddy):needs=X11 section=Games/Toys/Teddies\
>> -  title="Teddy" icon="teddy.xpm" command="/usr/bin/X11/teddy"
>> +  title="Teddy" icon="/usr/X11R6/include/X11/pixmaps/teddy.xpm"
>> command="/usr/bin/X11/teddy"
>>  ?package(xteddy):needs=X11 section=Games/Toys/Teddies\
>> -  title="XAlex" icon="xalex.xpm" command="/usr/bin/X11/xalex"
>> +  title="XAlex" icon="/usr/X11R6/include/X11/pixmaps/xalex.xpm"
>> command="/usr/bin/X11/xalex"
>>  ?package(xteddy):needs=X11 section=Games/Toys/Teddies\:.,$d
>> ...
> 
> 


-- 
To UNSUBSCRIBE, email to debian-devel-request@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org



Reply to: