Effect.SlideUp is too slow--next statement executes immediately

[nolo contendere]

the alert message appears before the Effect.SlideUp even begins. How
can I ensure that the SlideUp completes before executing the next
statement? I've tried setTimeout, and I can kind of get it to work,
but that seems very kludgy, and may not work in all cases.

TIA


=====================
function add_entry( id ) {
var row = $(id).insertRow( journal_rows++ );
var cell0 = row.insertCell(0);
cell0.width="150px";
cell0.innerHTML = "<div id=div_entry" + journal_rows + "
style='display: none;' >Journal Entry " + journal_rows + ":<img src='/
images/notes.gif'>" +
"<textarea id=txt" + journal_rows + " name=txt"
+ journal_rows + " cols=64 rows=5 ></textarea></div>";
Effect.SlideDown('div_entry' + journal_rows);
}

function delete_entry( id, force ) {
if ( journal_rows < 2 ) {
return;
}
if ( force ) {
$(id).deleteRow( --journal_rows );
}
else {
if ( confirm('want to delete journal row: ' + journal_rows +
'?' ) ) {
// The delete happens too fast, tried
setTimeout("deleteRow", 1000), but breaks...
Effect.SlideUp('div_entry' + journal_rows);
alert( 'id: ' + id);
$(id).deleteRow(--journal_rows);
// setTimeout("$(id).deleteRow(--journal_rows)", 2000);
}
}
}

 

[s0lnic]

Effect.SlideUp('div_entry' + journal_rows);

Use afterFinish callback

 

[SAM]

nolo contendere a écrit :

the alert message appears before the Effect.SlideUp even begins.

What is Effect.SlideUp() ?
Where is it defined ?
Does it return something in end of job ?

How can I ensure that the SlideUp completes before
executing the next statement?

giving a new special argument to Effect.SlideUp() ?
to fire in end of slide the row's deleting.

I've tried setTimeout, and I can kind of get it to work,
but that seems very kludgy, and may not work in all cases.

=====================
function add_entry( id ) {
var row = $(id).insertRow( journal_rows++ );
var cell0 = row.insertCell(0);
cell0.width="150px";
cell0.innerHTML = "<div id=div_entry" + journal_rows + "
style='display: none;' >Journal Entry " + journal_rows + ":<img src='/
images/notes.gif'>" +
"<textarea id=txt" + journal_rows + " name=txt"
+ journal_rows + " cols=64 rows=5 ></textarea></div>";
Effect.SlideDown('div_entry' + journal_rows);
}

function delete_entry( id, force ) {
if ( journal_rows < 2 ) {
return;
}
if ( force ) {
$(id).deleteRow( --journal_rows );
}

else {
if (confirm('want to delete journal row: '+journal_rows+'?') ) {
Effect.SlideUp('div_entry' + journal_rows);
setTimeout( function() {
$(id).deleteRow(--journal_rows);
}, 2000);
}
}
}

or ... :

else {
if(confirm('want to delete journal row: '+journal_rows+'?') ) {
Effect.SlideUp('div_entry' + journal_rows);
var this_journal_row = --journal_rows;
setTimeout('$('+id+').deleteRow('+this_journal_row+')', 2000);
}
}
}


but ... perhaps ... :

else {
if (confirm('want to delete journal row: '+journal_rows+'?') ) {
while(!Effect.SlideUp('div_entry' + journal_rows)) '';
$(id).deleteRow(--journal_rows);
}
}
}

 

[s0lnic]

nolo contendere a écrit :

What is  Effect.SlideUp() ?
Where is it defined ?
Does it return something in end of job ?

It's from script.aculo.us JS effects library.

 

[SAM]

s0lnic a écrit :

It's from script.aculo.us JS effects library.

Ha! OK. (50ko of compressed code ...!)

So the answer isn't it here :
<http://wiki.script.aculo.us/scriptaculous/show/EffectQueues>
?

 

[nolo contendere]

Use afterFinish callback

This worked! Thanks so much!

Effect.SlideUp('div_entry' + journal_rows, { afterFinish:
function () { $(id).deleteRow(--journal_rows) } } );

 

[nolo contendere]

s0lnic a écrit :



Ha! OK. (50ko of compressed code ...!)

So the answer isn't it here :
<http://wiki.script.aculo.us/scriptaculous/show/EffectQueues>
?

I believe that would have worked, although I didn't test it. I used
the afterFinish callback suggested by s01nic.

 

[Thomas 'PointedEars' Lahn]

It's from script.aculo.us JS effects library.

And hopefully the fact that at least two regulars of this newsgroup did not
know that has caused both of you to think in the right direction for a change.


PointedEars