From: SourceForge.net <no...@so...> - 2013-03-19 21:24:51
|
Bugs item #3608074, was opened at 2013-03-14 12:42 Message generated for change (Comment added) made by jenglish You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=112997&aid=3608074&group_id=12997 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: 01. Bindings Group: None >Status: Closed >Resolution: Invalid Priority: 5 Private: No Submitted By: Talvo (talvo) Assigned to: Joe English (jenglish) Summary: <<Invoke>> binding in lib/ttk/button.tcl Initial Comment: In lib/ttk/button.tcl, there are two bindings (which are later copied to TRadiobutton and TCheckbutton as well): bind TButton <Key-space> { ttk::button::activate %W } bind TButton <<Invoke>> { ttk::button::activate %W } <<Invoke>> is undocumented (it's not on the [event] manpage's list of Pre-Defined Events, or mentioned on the [ttk::button] manpage) - I'm not sure if this is an oversight, or if the binding is only there for backwards compatability or something. But it looks like it should probably either be removed, or should be documented (and the <Key-space> binding changed to [event add <<Invoke>> <Key-space>] so it's used). ---------------------------------------------------------------------- >Comment By: Joe English (jenglish) Date: 2013-03-19 14:24 Message: <<Invoke>> event bindings are used for mnemonic activation and for accelerators, for exaqmple: bind $dlg <KeyPress-Escape> [list event generate $dlg.cancel <<Invoke>>] See also the keynav package (which was a part of Tile that Jeff left behind when executing TIP#248), https://github.com/jenglish/tile-extras/blob/master/keynav.tcl Please do not remove the <<Invoke>> binding from library/ttk/*.tcl, that will break user code that relies on it. Do not add <Key-space> as a synonym for <<Invoke>>, that would be incorrect. ---------------------------------------------------------------------- Comment By: Jan Nijtmans (nijtmans) Date: 2013-03-19 07:36 Message: <<Invoke>> cannot be added to the virtual events, at least not until Tk 8.7. I see no added value in defining a virtual event which is equivalent to <Key-space>, so I would suggest removing it. Joe? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=112997&aid=3608074&group_id=12997 |