From 96b96cbc698373dfb42979323fdefafd2a627a88 Mon Sep 17 00:00:00 2001
From: Dan Walkowski <dwalkowski@mozilla.com>
Date: Wed, 31 Mar 2010 16:16:42 -0700
Subject: [PATCH] cleaned up drag/drop code

---
 browser/base/content/tabcandy/app/groups.js | 45 +++++----------------
 1 file changed, 9 insertions(+), 36 deletions(-)

diff --git a/browser/base/content/tabcandy/app/groups.js b/browser/base/content/tabcandy/app/groups.js
index 933db2b2f25c..6b6088ee8dd7 100644
--- a/browser/base/content/tabcandy/app/groups.js
+++ b/browser/base/content/tabcandy/app/groups.js
@@ -469,46 +469,19 @@ window.Groups = {
     tolerance: "pointer",
     greedy: true,
     drop: function(e){
-      $target = $(e.target);
+    $target = $(e.target);  
+    $dragged.removeClass("willGroup")   
   
-      // Only drop onto the top z-index
-      if( $target.css("zIndex") < $dragged.data("topDropZIndex") ) return;
-      $dragged.data("topDropZIndex", $target.css("zIndex") );
-      $dragged.data("topDrop", $target);
-      
-      // This strange timeout thing solves the problem of when
-      // something is dropped onto multiple potential drop targets.
-      // We wait a little bit to see get all drops, and then we have saved
-      // the top-most one and drop onto that.
-      clearTimeout( timeout );
-      var dragged = $dragged;
-      var target = $target;
-      timeout = setTimeout( function(){
-        dragged.removeClass("willGroup")   
-  
-        dragged.animate({
-          top: target.position().top+15,
-          left: target.position().left+15,      
-        }, 100);
-        
-        setTimeout( function(){
-          var group = $(target).data("group");
-          if( group == null ){
-            var group = new Group([target, dragged]);
-          } else {
-            group.add( dragged );
-          }
-          
-        }, 100);
-        
-        
-      }, 10 );
-      
+    var group = $target.data("group");
+    if( group == null ){
+      var group = new Group([$target, $dragged]);
+    } else {
+      group.add( $dragged );
+    }
       
     },
     over: function(e){
-      $dragged.addClass("willGroup");
-      $dragged.data("topDropZIndex", 0);    
+      $dragged.addClass("willGroup");    
     },
     out: function(){      
       $dragged.removeClass("willGroup");