don't set window bounds when TM is in sidepanel cos it gets the paren…

…t windows dimensions. Also changes for 1.2.1 were missed from sidepanel

extension/background.js

@@ -385,6 +385,7 @@ chrome.windows.onFocusChanged.addListener(logEventWrapper("windows.onFocusChange
 
 chrome.windows.onBoundsChanged.addListener(async (window) => {
     // remember position of topic manager window
+    if (BTManagerHome === 'SIDEPANEL') return;          // doesn't apply
     const [BTTab, BTWin] = await getBTTabWin();
     if (BTWin != window.id) return;
     const location = {top: window.top, left: window.left, width: window.width, height: window.height};

extension/sidePanel.js

@@ -18,8 +18,13 @@ let BTPort;
     const contentScripts = manifest.content_scripts;
     const match = contentScripts[0].matches[0];
     const localhost = match.includes('localhost');
-    const version = manifest.version;
+    let version = manifest.version;
+    // version is x.y.z or x.y, need to strip off the .z if present
+    const parts = version.split('.');
+    if (parts.length > 2) version = parts.slice(0, 2).join('.');
+
     const url = match.replace(/\*+$/, '') + (localhost ? '' : (version + '/app'));
+
     const iframe = document.getElementById('BTTopicManager');
     iframe.src = url;
 })();

versions/1.2/extension/sidePanel.js

@@ -18,8 +18,13 @@ let BTPort;
     const contentScripts = manifest.content_scripts;
     const match = contentScripts[0].matches[0];
     const localhost = match.includes('localhost');
-    const version = manifest.version;
+    let version = manifest.version;
+    // version is x.y.z or x.y, need to strip off the .z if present
+    const parts = version.split('.');
+    if (parts.length > 2) version = parts.slice(0, 2).join('.');
+
     const url = match.replace(/\*+$/, '') + (localhost ? '' : (version + '/app'));
+
     const iframe = document.getElementById('BTTopicManager');
     iframe.src = url;
 })();