From 924bcfab629fc7a3e13ba7070591da4d0a922c7a Mon Sep 17 00:00:00 2001
From: Chris <chris@chrismatic.io>
Date: Mon, 16 Mar 2015 17:40:17 -0600
Subject: [PATCH] Safari: remove hardcoded values from popup resizing

---
 platform/safari/vapi-popup.js | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/platform/safari/vapi-popup.js b/platform/safari/vapi-popup.js
index b875657b2..ce86ea246 100644
--- a/platform/safari/vapi-popup.js
+++ b/platform/safari/vapi-popup.js
@@ -21,22 +21,29 @@
 (function() {
 'use strict';
 
+var DF_ENABLED_CLASS = "dfEnabled";
+
 var onLoaded = function() {
-    var style = document.createElement("style");
-    style.textContent = "html,body,#panes{width:100%}#panes{white-space:nowrap;text-align:right}#panes > div:nth-of-type(2){display:inline-block !important}";
     var _toggle = DOMTokenList.prototype.toggle;
-    DOMTokenList.prototype.toggle = function(cls, stt) {
+    DOMTokenList.prototype.toggle = function(className) {
         _toggle.apply(this, arguments);
-        if(cls === "dfEnabled") {
-            isThere = stt;
+        if(className === DF_ENABLED_CLASS) {
             setTimeout(updateSize, 0);
         }
     };
     var body = document.body, popover = safari.self;
+    
+    var style = document.createElement("style");
+    style.textContent = "html,body,#panes{width:100%}#panes{white-space:nowrap;text-align:right}#panes > div:nth-of-type(2){display:inline-block !important}";
     body.appendChild(style);
-    var isThere = !!document.querySelector(".dfEnabled");
+
+    var panes = document.getElementById("panes"),
+        powerAndStatsPane = panes.children[0],
+        dfPane = panes.children[1];
+
     var updateSize = function() {
-        popover.width = 152 + (isThere ? 320 : 0);
+        var dfEnabled = panes.classList.contains(DF_ENABLED_CLASS);
+        popover.width = powerAndStatsPane.clientWidth + (dfEnabled ? dfPane.clientWidth : 0);
         popover.height = body.clientHeight;
     };
     updateSize();