wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1007601 Fix styling of the loop panel so it is laid out properly and without a white border. Also make the iframe creation dynamic, to avoid an iframe in the DOM when we don't need one. r=mhammond

Browse Source
This commit is contained in:
Mark Banner 2014-05-29 21:13:47 +01:00
parent 01b7e11bcb
commit 0f0d05150c
2 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
browser/base/content/browser-loop.js
View File

@ -23,6 +23,16 @@ XPCOMUtils.defineLazyModuleGetter(this, "MozLoopService", "resource:///modules/l
let anchor = event.target;
let iframe = document.getElementById("loop-panel-frame");
if (!iframe) {
// XXX This should be using SharedFrame (bug 1011392 may do this).
iframe = document.createElement("iframe");
iframe.setAttribute("id", "loop-panel-frame");
iframe.setAttribute("type", "content");
iframe.setAttribute("class", "loop-frame social-panel-frame");
iframe.setAttribute("flex", "1");
panel.appendChild(iframe);
}
// We inject in DOMContentLoaded as that is before any scripts have tun.
iframe.addEventListener("DOMContentLoaded", function documentDOMLoaded() {
iframe.removeEventListener("DOMContentLoaded", documentDOMLoaded, true);
@ -35,7 +45,7 @@ XPCOMUtils.defineLazyModuleGetter(this, "MozLoopService", "resource:///modules/l
iframe.contentWindow.removeEventListener("loopPanelInitialized",
documentLoaded, true);
// XXX We end up with the wrong size here, so this
// needs investigation (bug 1007601).
// needs further investigation (bug 1011394).
sizeSocialPanelToContent(panel, iframe);
}, true);

8
browser/base/content/browser.xul
View File

@ -269,14 +269,10 @@
#ifdef MOZ_LOOP
<panel id="loop-panel"
class="loop-panel"
class="loop-panel social-panel"
type="arrow"
orient="horizontal"
hidden="true">
<iframe id="loop-panel-frame"
type="content"
/>
</panel>
hidden="true"/>
#endif
<menupopup id="toolbar-context-menu"