Author: taylor
Date: Thu Nov 19 00:13:58 2009
New Revision: 882012

URL: http://svn.apache.org/viewvc?rev=882012&view=rev
Log:
https://issues.apache.org/jira/browse/JS2-1084
- fixing recently broken move-portlet feature bug. When drag source handled 
drag:start event, the source node was not being properly replaced by a gray box

Modified:
    
portals/jetspeed-2/portal/trunk/applications/jetspeed/src/main/webapp/jetui/engine/portal.js

Modified: 
portals/jetspeed-2/portal/trunk/applications/jetspeed/src/main/webapp/jetui/engine/portal.js
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/applications/jetspeed/src/main/webapp/jetui/engine/portal.js?rev=882012&r1=882011&r2=882012&view=diff
==============================================================================
--- 
portals/jetspeed-2/portal/trunk/applications/jetspeed/src/main/webapp/jetui/engine/portal.js
 (original)
+++ 
portals/jetspeed-2/portal/trunk/applications/jetspeed/src/main/webapp/jetui/engine/portal.js
 Thu Nov 19 00:13:58 2009
@@ -420,15 +420,11 @@
         }
         if (drag.get('node').data.get("toolbar"))
         {
-            drag.get('node').setStyle('visibility', '');
+            drag.get('node').setStyle('visibility', '');               
         }
         else
         {
-            var x = drag.get('node').all('div');
-            if (x != null)
-            {
-               x.setStyle('visibility', '');
-            }
+               drag.get('node').get('children').setStyle('visibility', '');
         }        
         //drag.get('node').setStyle('border', '');                
         drag.get('node').removeClass('moving');
@@ -473,16 +469,9 @@
     
     Y.DD.DDM.on('drag:start', function(e) {
         var drag = e.target;
-//        if (drag.target) {
-//            drag.target.set('locked', true);
-//        }
         var dragNode = drag.get('dragNode');
-        dragNode.set('innerHTML', drag.get('node').get('innerHTML'));
-        Y.log("class is " + drag.get('node').getAttribute('class'));
-        Y.log("node is " + drag.get('node').getAttribute('id'));
-
-        //        dragNode.setStyle('opacity','.5');
-//        dragNode.setAttribute('class', 'portlet purpleplanet');
+        var srcNode = drag.get('node');
+        dragNode.set('innerHTML', srcNode.get('innerHTML'));
 
         if (drag.get('node').data.get("toolbar"))
         {
@@ -490,15 +479,10 @@
         }
         else
         {
-               var x = drag.get('node').all('div');
-               if (x != null)
-               {
-                       x.setStyle('visibility', 'hidden');
-               }
+               srcNode.get('children').setStyle('visibility', 'hidden');
+            srcNode.addClass('moving');                
         }
-        drag.get('node').setStyle('border', '1px dotted #black');        
-        drag.get('node').addClass('moving');
-        
+        //  drag.get('node').setStyle('border', '1px dotted #black');        
         lastX = drag.mouseXY[0];
         lastY = drag.mouseXY[1];
         //Y.log("starting drag " + lastX +  " , " + lastY);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to