[openbox] window cycling dialog

[Mikael Magnusson]

On 3 August 2011 13:00, David Vogt <dv at adfinis.ch> wrote:
> Hi!
>
> On 03/08/11 12:00, Roman Pořízka wrote:
>>
>> Hello,
>> I have tried openbox 3.5 today and I've been using the least overlap
>> patch, but I cannot switch of the window cycling dialog... In my rc.xml
>> is:
>
> I don't think that the least-overlap patch has something to do with it. But:
>
>>
>> <keybind key="A-Tab">
>> <action name="NextWindow"/>
>> <dialog>no</dialog>
>> </keybind>
>
> As of my understanding, the <dialog> should be places inside the action,
> like this:
>
>    <keybind key="A-Tab">
>      <action name="NextWindow">
>        <dialog>no</dialog>
>      </action>
>    </keybind>
>
> However, this doesn't work for me either.
>
>> [...]
>>
>> Could you please tell me, what am I doing wrong?
>
> I think you found a bug - the <dialog>no</dialog> also fails in the
> SendToDesktop* actions, for example.
>
> Or maybe this is a feature that was removed intentionally? Maybe someone of
> the main developers could throw in a word?

I think it's a minor mistake. When we added the listmode, the value
changed from a boolean, and the new code looks for the word "none",
not "no", to disable the dialog.

-- 
Mikael Magnusson