PDA

View Full Version : [UNKNOWN][3.0.3] TreeEditor editDelay bug



Eric24
30 Nov 2009, 1:37 PM
The implementation of editDelay in 3.0.3 (and all prior versions) does't do what the docs say it does:


this.autoEditTimer = this.startEdit.defer(this.editDelay, this, [node.ui.textNode, value]);This was brought up a long time ago (http://www.extjs.com/forum/showthread.php?t=19911), but that thread went nowhere.

The current functionality is just about useless. It pretty much ensures that the user will mistakenly trigger the editor while they are navigating the tree. On top of this, there is a "functional race" condition between editing and expanding when double-clicking on the node (this has also been brought up before, with the answer being various overrides to disable one behavior or the other).

Since it's really not practical to have both behaviors active at the same time, I feel like the configs for the TreeEditor and/or the TreeNodeUI should provide an easy way to enable/disable their respective behaviors (on top of fixing the functionality of "editDelay", which would probably be best accomplished by eliminating it altogether in favor of basic a dblclick event).