You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jetspeed-dev@portals.apache.org by ta...@apache.org on 2009/11/19 03:27:03 UTC

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

Author: taylor
Date: Thu Nov 19 02:27:01 2009
New Revision: 882030

URL: http://svn.apache.org/viewvc?rev=882030&view=rev
Log:
https://issues.apache.org/jira/browse/JS2-1084
- deletes were failing due to "." in element ids conflicting with css selector api. escaped . and # in selectors

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=882030&r1=882029&r2=882030&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 02:27:01 2009
@@ -231,11 +231,18 @@
         }
     });
 
+    var cleanseForSelector = function(s) {
+    	return s.replace(/(\.|\#)/g, "\\$1");
+    }
+
+    Y.log(" *** replacing " + cleanseForSelector("widget.name#address.cool#3"));
+    
     var onRemoveComplete = function(id, o, args) { 
     	var id = id; // Transaction ID. 
     	var data = o.responseText; // Response data. 
     	var widgetId = args[0];
-    	// now remove it from the dom
+    	// widgetId = widgetId.replace("\.", "\\.");
+    	widgetId = cleanseForSelector(widgetId);
     	var widget = Y.one("#" + widgetId);
     	if (widget)
     	{
@@ -243,7 +250,6 @@
     		widget.remove();
 	        if (parent.get('children').size() == 0)
 	        {
-	        	//node.plug(Y.Plugin.Drag);
 		    	var drop = new Y.DD.Drop({
 		        node: parent,
 		        groups: ['portlets']            
@@ -547,11 +553,11 @@
 		    	var srcRegion = e.drag.get('node').get('region');
 		    	if (y >= srcRegion.top && y <= srcRegion.bottom && x >= srcRegion.left && x <= srcRegion.right)
 		    	{
-	//	    		Y.log("dragging over src");
+		    		//Y.log("dragging over src");
 		    	}	    	
 		    	else if (y >= region.top && y <= region.bottom && x >= region.left && x <= region.right)
 		    	{
-		    		//Y.log("**** HIT");
+		    		// Y.log("**** HIT");
 		    		portal.isMoving = true;
 		    		portal.movePortlet(e); 
 		    		portal.isMoving = false;



---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org