[Tcl-bugs] [ tktoolkit-Bugs-577906 ] %P substitution not documented

[SourceForge.net <no...@so...>]

Bugs item #577906, was opened at 2002-07-05 20:45
Message generated for change (Comment added) made by dkf
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112997&aid=577906&group_id=12997

Category: 65. Generic Window Operations
Group: = 8.4b1
Status: Open
Resolution: None
Priority: 9
Submitted By: Don Porter (dgp)
Assigned to: Joe English (jenglish)
Summary: %P substitution not documented

Initial Comment:


TIP 47 allegedly added a %P substitution to [bind] scripts,

but it is not mentioned in doc/bind.n





----------------------------------------------------------------------

>Comment By: Donal K. Fellows (dkf)
Date: 2003-09-07 17:37

Message:
Logged In: YES 
user_id=79902

"and" as you are selecting from a list (and that's the style

used elsewhere in the paragraph, which is my real criterion

there!)  However, your longer version is better still.  :^)



Clarifying %s for other situations is a nice longer-term

goal, but we're probably justified in ignoring it within the

scope of this bugfix.

----------------------------------------------------------------------

Comment By: Joe English (jenglish)
Date: 2003-09-07 17:28

Message:
Logged In: YES 
user_id=68433

"one of the strings _NewValue_, and _Delete_ is

substituted", or "one of the strings _NewValue_ or _Delete_

is substituted"?



Hm.



How about: "For _PropertyNotify_ events, substituted with

either the string _NewValue_ (indicating that the property

has been created or modified) or _Delete_ (indicating that

the property has been removed)"



... and while we're on the subject of the '%s' substitution,

probably ought to clarify the meaning of the "decimal

string" which is substituted for ButtonPress et. al. 

----------------------------------------------------------------------

Comment By: Donal K. Fellows (dkf)
Date: 2003-09-07 16:50

Message:
Logged In: YES 
user_id=79902

I prefer the following wording as being more in keeping with

the rest of the manpage and perhaps a little bit less obscure:

  For \fBProperty\fP events, one of the strings

\fBNewValue\fP, and \fBDelete\fP is substituted.

...

  The name of the property being updated or deleted (which

may be converted to an XAtom using \fBwinfo atom\fR.) Valid

only for \fBProperty\fP events.

Aside from that, the patch is good for application to both

the HEAD and the 8.4 branch (given that the %P substitution

was introduced as part of TIP#47.)  Boosting priority...

----------------------------------------------------------------------

Comment By: Joe English (jenglish)
Date: 2003-09-01 18:15

Message:
Logged In: YES 
user_id=68433

Must have missed that one.  Attached patch rectifies.



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112997&aid=577906&group_id=12997