From 8723504185cec53e8fbecbbeed9f74881d4b5e42 Mon Sep 17 00:00:00 2001 From: real_nowhereman Date: Mon, 23 Aug 2010 14:23:49 +0000 Subject: [PATCH] partial fix to widget panel bug (still error on trashing --IE8) --- admin/panels/widgets/admin.widgets.js | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/admin/panels/widgets/admin.widgets.js b/admin/panels/widgets/admin.widgets.js index 4eb859e..da9a9d5 100644 --- a/admin/panels/widgets/admin.widgets.js +++ b/admin/panels/widgets/admin.widgets.js @@ -45,8 +45,9 @@ function wplaceholder() { $(this).animate({'background-color' : '#fff'}) }, 'drop' : function(event, ui) { - parent=ui.draggable.parent(); - where=$(this).parent().attr('id').split('-')[1]; + var parent=ui.draggable.parent(); + var where=$(this).parent().attr('id').split('-')[1]; + var replace = null; if(ui.draggable.hasClass('widget-instance')) { replace=ui.draggable; } else { @@ -72,8 +73,9 @@ function winstancedrop() { $(this).animate({'background-color' : '#fff'}) }, 'drop' : function(event, ui) { - parent=ui.draggable.parent(); - where=$(this).parent().attr('id').split('-')[1]; + var parent=ui.draggable.parent(); + var where=$(this).parent().attr('id').split('-')[1]; + var replace = null; if(ui.draggable.hasClass('widget-instance')) { replace=ui.draggable; } else { @@ -100,8 +102,9 @@ function wtrash() { $(this).animate({'background-color' : '#a22'}) }, 'drop' : function(event, ui) { - parent=ui.draggable.parent(); - $(ui.draggable).fadeOut().remove();; + var parent=ui.draggable.parent(); + $(ui.draggable).fadeOut().remove(); // error in IE8 + // $(ui.draggable).fadeOut(); // will fade, will remove, graphic glitches $('.widget-dragger').remove(); if(parent.children().length<1) { parent.append('
  • Drop here
  • '); @@ -118,4 +121,4 @@ function wreload(){ winstancedrop(); wtrash(); } -$(document).ready(wreload); \ No newline at end of file +$(document).ready(wreload);