Author: lindner
Date: Sat Apr 24 02:02:33 2010
New Revision: 937575

URL: http://svn.apache.org/viewvc?rev=937575&view=rev
Log:
add placeholder for future popup view support

Modified:
    shindig/trunk/features/src/main/javascript/features/views/views.js

Modified: shindig/trunk/features/src/main/javascript/features/views/views.js
URL: 
http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/views/views.js?rev=937575&r1=937574&r2=937575&view=diff
==============================================================================
--- shindig/trunk/features/src/main/javascript/features/views/views.js 
(original)
+++ shindig/trunk/features/src/main/javascript/features/views/views.js Sat Apr 
24 02:02:33 2010
@@ -286,6 +286,9 @@ gadgets.views = function() {
       if (typeof view !== "string") {
         view = view.getName();
       }
+      // TODO If we want to implement a POPUP view we'll have to do it here,
+      // The parent frame's attempts to use window.open will fail since it's 
not
+      // directly initiated from the onclick handler
       gadgets.rpc.call(null, "requestNavigateTo", null, view, opt_params, 
opt_ownerId);
     },
 


Reply via email to