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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to