Scott --

On Fri, 28 Jul 2006 15:27:22 +1000 Scott Smedley <[EMAIL PROTECTED]> wrote:

> Hi Thomas,
> 
> > I noticed earlier that there is a discrepency in the FVWM manpage (2.5.X
> > series is the only one to be fixed) for the Wait command description.  The
> > description says initially that the command waits of "windowname".   This
> > is parly true -- the code in builtins.c apparently also checks for the
> > window's tile, class and resource (just like how the style command works).
> > 
> > I feel this needs to be reflected in its description
> 
> ...
> 
> > +.BI "Wait [" windowtitle | windowclass | windowresource "]"
> 
> Personally, I think it would be prudent to stick with "windowname" here.
> A novice user might think it appropriate to specify, say, a window resource
> & then be surprised to learn that some other window with a matching
> _title_ was matched.


That's fine, and it makes sense of course.  Quite often a window's name (as
it appears on the titlebar) is taken from WM_TITLE anyway.


> How about:
> 
> Wait windowname
>       This  command is intended to be used in fvwm functions only.  It
>       causes execution of a function to pause until a new window matching
>       windowname appears. A window can match windowname on either its
>       title, class or resource. This is particularly useful in
>       "InitFunction" if you are trying to start windows on specific desks
>       or pages:
> 
> It's still not perfect - any suggested improvements?


That's better.  I still think it is wise to specify in the syntax for the
command the fact that one can have:  windowname, windowclass, windowresource
-- and while I realise this forces a slight amount of Xlib knowledge onto the
user, it does clarify the list of options one can use to that command.  When
I first read it I was confused, since it did (to  me) read as though the
window's name was only considered.


> > Note that in the function example accompanying the Wait command's
> > description, I have taken the liberty of replacing the function's use of
> > the "Desk" command with "GotoDesk" since this is now the preferred
> > command to use.
> 
> Ok.
> 
> > ThisWindow (thomas_adam) Destroy
> 
> ThisWindow (thomas_adam) Wait godo :)


Heh.

-- Thomas Adam

-- 
ThisWindow (thomas_adam) Destroy

Reply via email to