[openbox] Quake-like-console configuration: problem with ForEach action

[Dana Jansens]

On Fri, Jan 2, 2015 at 1:05 PM, Martin Jiřička <mjiricka at gmail.com> wrote:

> > It feels like there may be a bug here though. If you put the tests in a
> > <query> with target="default" then does it work?
>
> No, it does not.
>
> But what is definitely buggy is this code:
>
>   <keybind key="W-x">
>     <action name="If">
>       <query target="default">
>         <title>quake_console</title>
>       </query>
>       <then>
>         <action name="SendToDesktop">
>           <to>3</to>
>           <wrap>no</wrap>
>           <follow>no</follow>
>         </action>
>       </then>
>       <else>
>         <action name="SendToDesktop">
>           <to>4</to>
>           <wrap>no</wrap>
>           <follow>no</follow>
>         </action>
>       </else>
>     </action>
>   </keybind>
>
> This sends all windows into Desktop 3, thus the condition is always
> evaluated as true.
>

Hm, something's definitely sketchy. Can you file a bug in the bug tracker
about it so it doesn't get lost?


> Moreover I do not understand why the ForEach action would need a
> "target" attribute because -- as I understand it -- it should test all
> windows.
>

It lets you act on all windows based on tests against your current focus
window.


> _______________________________________________
> openbox mailing list
> openbox at icculus.org
> http://icculus.org/mailman/listinfo/openbox
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://icculus.org/pipermail/openbox/attachments/20150102/20b954a9/attachment.html>