X-Git-Url: http://git.ithinksw.org/extjs.git/blobdiff_plain/530ef4b6c5b943cfa68b779d11cf7de29aa878bf..7a654f8d43fdb43d78b63d90528bed6e86b608cc:/src/dd/DropTarget.js

diff --git a/src/dd/DropTarget.js b/src/dd/DropTarget.js
index c1fddfc1..c3715f7f 100644
--- a/src/dd/DropTarget.js
+++ b/src/dd/DropTarget.js
@@ -1,9 +1,3 @@
-/*!
- * Ext JS Library 3.2.1
- * Copyright(c) 2006-2010 Ext JS, Inc.
- * licensing@extjs.com
- * http://www.extjs.com/license
- */
 /**
  * @class Ext.dd.DropTarget
  * @extends Ext.dd.DDTarget
@@ -13,21 +7,23 @@
  * @param {Mixed} el The container element
  * @param {Object} config
  */
-Ext.dd.DropTarget = function(el, config){
-    this.el = Ext.get(el);
-    
-    Ext.apply(this, config);
-    
-    if(this.containerScroll){
-        Ext.dd.ScrollManager.register(this.el);
-    }
-    
-    Ext.dd.DropTarget.superclass.constructor.call(this, this.el.dom, this.ddGroup || this.group, 
-          {isTarget: true});
+Ext.define('Ext.dd.DropTarget', {
+    extend: 'Ext.dd.DDTarget',
+    requires: ['Ext.dd.ScrollManager'],
+
+    constructor : function(el, config){
+        this.el = Ext.get(el);
+
+        Ext.apply(this, config);
+
+        if(this.containerScroll){
+            Ext.dd.ScrollManager.register(this.el);
+        }
 
-};
+        this.callParent([this.el.dom, this.ddGroup || this.group,
+              {isTarget: true}]);
+    },
 
-Ext.extend(Ext.dd.DropTarget, Ext.dd.DDTarget, {
     /**
      * @cfg {String} ddGroup
      * A named drag drop group to which this object belongs.  If a group is specified, then this object will only
@@ -41,12 +37,12 @@ Ext.extend(Ext.dd.DropTarget, Ext.dd.DDTarget, {
      * @cfg {String} dropAllowed
      * The CSS class returned to the drag source when drop is allowed (defaults to "x-dd-drop-ok").
      */
-    dropAllowed : "x-dd-drop-ok",
+    dropAllowed : Ext.baseCSSPrefix + 'dd-drop-ok',
     /**
      * @cfg {String} dropNotAllowed
      * The CSS class returned to the drag source when drop is not allowed (defaults to "x-dd-drop-nodrop").
      */
-    dropNotAllowed : "x-dd-drop-nodrop",
+    dropNotAllowed : Ext.baseCSSPrefix + 'dd-drop-nodrop',
 
     // private
     isTarget : true,
@@ -66,7 +62,7 @@ Ext.extend(Ext.dd.DropTarget, Ext.dd.DDTarget, {
      */
     notifyEnter : function(dd, e, data){
         if(this.overClass){
-            this.el.addClass(this.overClass);
+            this.el.addCls(this.overClass);
         }
         return this.dropAllowed;
     },
@@ -95,7 +91,7 @@ Ext.extend(Ext.dd.DropTarget, Ext.dd.DDTarget, {
      */
     notifyOut : function(dd, e, data){
         if(this.overClass){
-            this.el.removeClass(this.overClass);
+            this.el.removeCls(this.overClass);
         }
     },
 
@@ -107,9 +103,16 @@ Ext.extend(Ext.dd.DropTarget, Ext.dd.DDTarget, {
      * @param {Ext.dd.DragSource} source The drag source that was dragged over this drop target
      * @param {Event} e The event
      * @param {Object} data An object containing arbitrary data supplied by the drag source
-     * @return {Boolean} True if the drop was valid, else false
+     * @return {Boolean} False if the drop was invalid.
      */
     notifyDrop : function(dd, e, data){
         return false;
+    },
+
+    destroy : function(){
+        this.callParent();
+        if(this.containerScroll){
+            Ext.dd.ScrollManager.unregister(this.el);
+        }
     }
-});
\ No newline at end of file
+});