| Issue 112: | Remove Function Errors under Persistence and First and only child node is removed | |
| 1 person starred this issue and may be notified of changes. | Back to list |
What steps will reproduce the problem?
1. Enable persistence
2. Expand node to view child node
3. Remove single child node that is expanded.
What version of the product are you using? .5
On what operating system and browser? xp ie7
Node is removed as expected, but persistence tries to re-expand node after
all child nodes were removed.
Simply change... approx line 719 error goes away.
This is obviously due to persistence trying to expand the child node that
was selected to be removed.
var fxDuration = opts.fx ? (opts.fx.duration || 200) : 0;
for(var i=0; i<this.childList.length; i++ ) {
var $child = $(this.childList[i].div);
if( fxDuration ) {
// This is a toggle, so only do it, if not
already rendered (in)visible ( issue 98 )
if( bExpand != $child.is(':visible') )
$child.animate(opts.fx,
fxDuration);
} else {
if( bExpand )
$child.show();
else
$child.hide(); // TODO: this seems
to be slow, when called the first time for an element
}
}
TO
var fxDuration = opts.fx ? (opts.fx.duration || 200) : 0;
if (this.childList)
{
for(var i=0; i<this.childList.length; i++ ) {
var $child = $(this.childList[i].div);
if( fxDuration ) {
// This is a toggle, so only do
it, if not already rendered (in)visible ( issue 98 )
if( bExpand != $child.is
(':visible') )
$child.animate(opts.fx,
fxDuration);
} else {
if( bExpand )
$child.show();
else
$child.hide(); // TODO:
this seems to be slow, when called the first time for an element
}
}
}
Please provide any additional information below.
Sep 2, 2009
no feedback given; closing this as 'fixed'
Status:
Fixed
Jul 17, 2012
considered verified
Status:
Verified
|
Labels: Milestone-Release0.5.1