You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by li...@apache.org on 2010/04/24 04:02:33 UTC
svn commit: r937575 -
/shindig/trunk/features/src/main/javascript/features/views/views.js
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);
},