Author: lindner
Date: Sat Dec 25 09:21:34 2010
New Revision: 1052763

URL: http://svn.apache.org/viewvc?rev=1052763&view=rev
Log:
SHINDIG-1479 | Patch from Andy Smith | several bugs in the common container 
test page

Modified:
    
shindig/trunk/features/src/main/javascript/features/container/gadget_holder.js

Modified: 
shindig/trunk/features/src/main/javascript/features/container/gadget_holder.js
URL: 
http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/container/gadget_holder.js?rev=1052763&r1=1052762&r2=1052763&view=diff
==============================================================================
--- 
shindig/trunk/features/src/main/javascript/features/container/gadget_holder.js 
(original)
+++ 
shindig/trunk/features/src/main/javascript/features/container/gadget_holder.js 
Sat Dec 25 09:21:34 2010
@@ -214,9 +214,19 @@ shindig.container.GadgetHolder.prototype
  * @private
  */
 shindig.container.GadgetHolder.prototype.doOaaIframeHtml_ = function() {
-  var iframeAttrs = {
-      frameborder: 'no',
-      scrolling: 'no'
+  var iframeParams = {
+      'id': this.iframeId_,
+      'name': this.iframeId_,
+      'src': this.getIframeUrl_(),
+      'scrolling': 'no',
+      'marginwidth': '0',
+      'marginheight': '0',
+      'frameborder': '0',
+      'vspace': '0',
+      'hspace': '0',
+      'class': this.renderParams_[shindig.container.RenderParam.CLASS],
+      'height': this.renderParams_[shindig.container.RenderParam.HEIGHT],
+      'width': this.renderParams_[shindig.container.RenderParam.WIDTH]
   };
   new OpenAjax.hub.IframeContainer(
       gadgets.pubsub2router.hub,
@@ -235,7 +245,7 @@ shindig.container.GadgetHolder.prototype
           parent: this.el_,
           uri: this.getIframeUrl_(),
           tunnelURI: shindig.uri('/gadgets/' + 
'../container/rpc_relay.html').resolve(shindig.uri(window.location.href)),
-          iframeAttrs: iframeAttrs
+          iframeAttrs: iframeParams
         }
       }
   );


Reply via email to