-
11 Feb 2009 9:35 AM #21
Insert not rolled back
Insert not rolled back
Hey - cool toy.
You need to insert the following at line 476
case 'insertChild':
so that it reads:
PHP Code:if(true !== o.success) {
this.showError(o.error || o.errors);
switch(options.action) {
case 'appendChild':
case 'insertChild':
options.node.parentNode.removeChild(options.node);
break;
default:
break;
}
return;
}
-
11 Feb 2009 9:41 AM #22
Node stays on error?
Jozef Sakalos, aka Saki
A lot of valuable info at:
Saki's Extensions and Plugins
Saki's Extensions and Plugins Docs
Saki's Examples, Latest: Grid in Card Layout
Saki's Blog, Featured: Writing a Big Application in Ext, Latest: Grid MultiSearch Plugin Video
-
11 Feb 2009 9:42 AM #23
Oh, yes, now I remember. I've added insertChild later and forgotten to put it here. Thank you for pointing out.
Jozef Sakalos, aka Saki
A lot of valuable info at:
Saki's Extensions and Plugins
Saki's Extensions and Plugins Docs
Saki's Examples, Latest: Grid in Card Layout
Saki's Blog, Featured: Writing a Big Application in Ext, Latest: Grid MultiSearch Plugin Video
-
25 Feb 2009 1:57 AM #24
just a suggestion
just a suggestion
I thiknk that a node.select when show context menu is a good approach because when i select a node and then right click to another for insert a new node the selected node stays until i insert the new node.
It might be confusing for some people
just a suggestion
Thanks is really great jobStill trying to Lear ExtJs
-
25 Feb 2009 2:35 AM #25
Node name you're working with is always displayed in bold as first non-editable item of context menu. However, users sometimes "do not see" so I've implemented your suggestion.
I've also released 1.0 stable version - there were no bugs reported so far, neither feature requests.Jozef Sakalos, aka Saki
A lot of valuable info at:
Saki's Extensions and Plugins
Saki's Extensions and Plugins Docs
Saki's Examples, Latest: Grid in Card Layout
Saki's Blog, Featured: Writing a Big Application in Ext, Latest: Grid MultiSearch Plugin Video
-
3 Mar 2009 3:58 AM #26
I've added some code to allow for editable: false
Please take a look and let me know if you think it's good enough to be included in your version.
w://
-
3 Mar 2009 11:51 AM #27
Could you please describe the changes, what's purpose of them, etc.?
Jozef Sakalos, aka Saki
A lot of valuable info at:
Saki's Extensions and Plugins
Saki's Extensions and Plugins Docs
Saki's Examples, Latest: Grid in Card Layout
Saki's Blog, Featured: Writing a Big Application in Ext, Latest: Grid MultiSearch Plugin Video
-
4 Mar 2009 2:21 AM #28
Sure.
I just changed it so that when 'editable: false' it:
- doesn't error
- shows a limited set of items in the context menu (only those items that do not pertain to editing).
If you do a diff on the file on your site to the one I uploaded you'll see they are really only minor changes but it makes sense to have this working as the property (editable) exists but the functionality didn't.
I have also extended this plugin to make it easier for people working with castle monorail bringing the following to the table:
- ability to specify controller class
- ability to specify actions for each operation (rename, add, delete etc...)
This means that we can have a seperate controller action for each tree operation - cleaner, simpler, easier to understand.
If anyone's interested I can upload.
Cheers for a cool plugin
w://
-
4 Mar 2009 11:54 AM #29
Oh, yes, editable:false is unfinished work of mine. I'll complete it soon. No idea what's "castle monorail" though.
Jozef Sakalos, aka Saki
A lot of valuable info at:
Saki's Extensions and Plugins
Saki's Extensions and Plugins Docs
Saki's Examples, Latest: Grid in Card Layout
Saki's Blog, Featured: Writing a Big Application in Ext, Latest: Grid MultiSearch Plugin Video
-
5 Mar 2009 4:27 AM #30
Hey
The file I uploaded has the editable: false functionality implemented so you can use that if you like
Castle Monorail is a shiz hot MVC for .NET - The whole Castle stack is worth checking out if you're a .NET dev:
http://www.castleproject.org/
w://


Reply With Quote
