[openbox] some bugs

[Dana Jansens]

Hi Vlad,

On Sun, Feb 3, 2008 at 10:28 AM, vlad <vla at uni-bonn.de> wrote:
>  first bug:
>  i've tried the <bar> and <raise> options with the NextWindow entries in
>  rc.xml. this leads to an openbox crash.
>  in my rc.xml i have following:
>     <keybind key="A-Tab">
>       <action name="NextWindow">
>       <bar>no</bar>
>       <raise>yes</raise>
>         <finalactions>
>           <action name="Focus"/>
>           <action name="Raise"/>
>         </finalactions>
>       </action>
>     </keybind>
>  now when pressing alt-tab ob crashes. sometimes it crashes directly,
>  sometimes after 2-3 times.

I can't reproduce this at all... there must be some other factor
involved.  Might I have a copy of your rc.xml and .xinitrc.  And/or
could you produce a backtrace for the crash?

>  second:
>  the client-combined-menu doesn't show the desktop names but only "manage
>  desktops" (see screenshot: http://img248.imageshack.us/img248/9868/screenshotfrom20080203afq2.png).
>  i don't think that's intended.

this was easy to track down and fix. thanks :)

>  now a feature request:
>  could you please make the "manage desktop"-stuff in the client-combined-menu optional.
>  for now i comment following lines in client_list_menu.c out:
>  lines 155-159:
>  "
>     /*
>     menu_add_separator(menu, SEPARATOR, NULL);
>     menu_add_normal(menu, ADD_DESKTOP, _("_Add new desktop"), NULL, TRUE);
>     menu_add_normal(menu, REMOVE_DESKTOP, _("_Remove last desktop"),
>                     NULL, TRUE);
>     */

this has also been done by mikachu so you should thank him :)


cheers,
dana