On Sat, Jul 03, 2010 at 05:02:40PM +0200, BOURDOISEAU wrote:
> Le vendredi 02 juillet 2010 à 19:48 +0200, Bill Allombert a écrit :
>
> > This is a bug in openbox menu-methods: it seems openbox needs the id field
> > to be unique, but the menu include two sections named 'Monitoring' and
> > the menu-method create the same id for both, hence the confusion.
> >
> > Please find a patch that change the id to be $section. This needs a small
> > change to /etc/xdg/openbox/menu.xml as well.
> >
> > Thanks for reporting this issue.
> >
> > Cheers,
>
>
> I modified files : /etc/menu-method/openbox and /etc/xdg/openbox/menu.xml
>
> I attached the diff and new files
>
> I am not on that it is the good solution, but it corrects the bug.
Ah I see I forgot to attach my patch... In any case, your patch
is as good as mine.
Cheers,
--
Bill. <[email protected]>
Imagine a large red swirl here.
--- openbox 2010-07-02 19:29:40.000000000 +0200
+++ /etc/menu-methods/openbox 2010-07-02 19:40:06.000000000 +0200
@@ -29,7 +29,7 @@
nstring(level(), " ") "</item>\n"
endsupported
-startmenu= nstring(level(), " ") "<menu id='" xml_escape(title()) "'
label='" xml_escape(title()) "'>\n"
+startmenu= nstring(level(), " ") "<menu id='" xml_escape($section) "'
label='" xml_escape(title()) "'>\n"
endmenu= nstring(level(), " ") "</menu>\n"
preoutput= "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n\n<!-- Automatically
generated file. Do not edit (see /usr/share/doc/menu/html) -->\n\n<openbox_menu
xmlns=\"http://openbox.org/\"\n
xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\"\n
xsi:schemaLocation=\"http://openbox.org/\n
file:///usr/share/openbox/menu.xsd\">\n\n"
--- menu.xml 2010-07-02 19:40:11.000000000 +0200
+++ /etc/xdg/openbox/menu.xml 2010-07-02 19:40:16.000000000 +0200
@@ -13,7 +13,7 @@
<action name="Execute"><execute>x-www-browser</execute></action>
</item>
<!-- This requires the presence of the 'menu' package to work -->
- <menu id="Debian" />
+ <menu id="/Debian" />
<separator />
<menu id="client-list-menu" />
<separator />
