From 33590aa353967f5a6162d7388e5db2a9aed1f03c Mon Sep 17 00:00:00 2001
From: Tamius Han <tamius.han@gmail.com>
Date: Sat, 15 Jun 2019 22:58:19 +0200
Subject: [PATCH] UX: In popup, indicate which sites (and corresponding frames)
 are disabled. This will need additional work on content scripts tho

---
 src/ext/lib/Settings.js |  2 +-
 src/popup/App.vue       | 93 +++++++++++++++++++++++++++++++++++++----
 src/res/css/common.scss |  4 ++
 3 files changed, 90 insertions(+), 9 deletions(-)

diff --git a/src/ext/lib/Settings.js b/src/ext/lib/Settings.js
index 91b6eba..19f28af 100644
--- a/src/ext/lib/Settings.js
+++ b/src/ext/lib/Settings.js
@@ -294,7 +294,7 @@ class Settings {
         return this.active.sites['@global'].mode === ExtensionMode.Enabled;
       }
 
-      if(this.active.sites['@global'].mode === ExtensionMode.Enabled) {
+      if (this.active.sites['@global'].mode === ExtensionMode.Enabled) {
         return this.active.sites[site].mode !== ExtensionMode.Disabled;
       } else if (this.active.sites['@global'].mode === ExtensionMode.Whitelist) {
         return this.active.sites[site].mode === ExtensionMode.Enabled;
diff --git a/src/popup/App.vue b/src/popup/App.vue
index 0d01dc3..67d3402 100644
--- a/src/popup/App.vue
+++ b/src/popup/App.vue
@@ -18,7 +18,7 @@
           </div>
         </div>
         <div class="menu-item"
-            :class="{'selected-tab': selectedTab === 'site'}"
+            :class="{'selected-tab': selectedTab === 'site', 'disabled': siteTabDisabled}"
             @click="selectTab('site')"
         >
           <div class="">
@@ -32,7 +32,10 @@
               <div v-for="site of activeSites"
                   :key="site.host"
                   class="tabitem"
-                  :class="{'tabitem-selected': site.host === selectedSite}"
+                  :class="{
+                    'tabitem-selected': site.host === selectedSite,
+                    'tabitem-disabled': !settings.canStartExtension(site.host)
+                  }"
                   @click="selectSite(site.host)"
               >
                 {{site.host}}
@@ -41,11 +44,11 @@
           </div>
         </div>
         <div class="menu-item"
-            :class="{'selected-tab': selectedTab === 'video'}"
+            :class="{'selected-tab': selectedTab === 'video', 'disabled': !canShowVideoTab.canShow}"
             @click="selectTab('video')"
         >
           <div class="">
-            Video settings
+            Video settings <span v-if="canShowVideoTab.canShow && canShowVideoTab.warning" class="warning-color">⚠</span>
           </div>
           <div v-if="selectedTab === 'video' && this.activeFrames.length > 0"
                class=""
@@ -54,7 +57,10 @@
             <div class="">
               <div v-for="frame of activeFrames"
                    class="tabitem"
-                   :class="{'tabitem-selected': selectedFrame === frame.id}"
+                   :class="{
+                     'tabitem-selected': selectedFrame === frame.id,
+                     'disabled': !isDefaultFrame(frame.id) && !settings.canStartExtension(frame.label)
+                   }"
                    :key="frame.id"
                    @click="selectFrame(frame.id)"
               >
@@ -164,6 +170,7 @@ import Settings from '../ext/lib/Settings';
 import ExecAction from './js/ExecAction.js';
 import DefaultSettingsPanel from './panels/DefaultSettingsPanel';
 import AboutPanel from './panels/AboutPanel';
+import ExtensionMode from '../common/enums/extension-mode.enum';
 
 export default {
   data () {
@@ -182,6 +189,30 @@ export default {
       currentZoom: 1,
       execAction: new ExecAction(),
       settings: new Settings(undefined, () => this.updateConfig()),
+      siteTabDisabled: false,
+      videoTabDisabled: false,
+    }
+  },
+  computed: {
+    canShowVideoTab() {
+      let canShow = false;
+      let warning = false;
+      let t;
+
+      if (!this.settings) {
+        return {canShow: true, warning: false};
+      }
+      for (const site of this.activeSites) {
+        t = this.settings.canStartExtension(site.host);
+        canShow = canShow || t;
+        warning = warning || !t;
+      }
+      if (t === undefined) {
+        // something isn't the way it should be. Show sites.
+        return {canShow: true, warning: true};
+      }
+
+      return {canShow, warning}
     }
   },
   async created() {
@@ -251,12 +282,12 @@ export default {
 
     },
     processReceivedMessage(message, port) {
-      if (Debug.debug) {
+      if (Debug.debug && Debug.comms) {
         console.log("[popup.js] received message set-c", message);
         console.log("[popup.js] message cloned set-c", JSON.parse(JSON.stringify(message)));
       }
 
-      if(message.cmd === 'set-current-site'){
+      if (message.cmd === 'set-current-site'){
         if (this.site) {
           if (!this.site.host) {
             // dunno why this fix is needed, but sometimes it is
@@ -280,6 +311,7 @@ export default {
 
         // loadConfig(site.host); TODO
         this.loadFrames(this.site);
+        this.showFirstTab(this.site);
       } else if (message.cmd === 'set-current-zoom') {
         this.setCurrentZoom(message.zoom);
       } else if (message.cmd === 'performance-update') {
@@ -288,7 +320,48 @@ export default {
         }
       }
 
-      return true;
+return true;
+    },
+    showFirstTab(videoTab) {
+      // determine which tab to show.
+      // Extension global disabled — show 'extension settings'
+      // Extension site disabled, no embedded videos — show 'site settings'
+      // Extension site disabled, embedded videos from non-blacklisted hosts — show video settings
+      // Extension site enabled — show vido settings
+
+      if (! this.settings.canStartExtension('@global')) {
+        if (this.selectedTab === 'video' || this.selectedTab === 'site') {
+          this.selectTab('global');
+        }
+        this.siteTabDisabled = true;
+        this.videoTabDisabled = true;
+        return;
+      }
+
+      this.siteTabDisabled = false;;
+      if (! this.settings.canStartExtension(this.site.host)) {
+        if (videoTab.frames.length > 1) {
+          for (const frame of videoTab.frames) {
+            if (this.settings.canStartExtension(frame.host)) {
+              this.videoTabDisabled = false;
+              // video is selected by default, so no need to reselect it
+              // and if video is not selected, the popup would switch to 'video'
+              // tab once every 5 seconds. We don't want that.
+              // this.selectTab('video');
+              return;
+            }
+          }
+        }
+        this.videoTabDisabled = true;
+        if (this.selectedTab === 'video') {
+          this.selectTab('site');
+        }
+        return;
+      }
+      this.videoTabDisabled = false;
+    },
+    isDefaultFrame(frameId) {
+      return frameId === '__playing' || frameId === '__all';
     },
     loadFrames(videoTab) {
       if (videoTab.selected) {
@@ -449,6 +522,10 @@ html, body {
   padding-right: 0.5em;
 }
 
+.tabitem-disabled {
+  color: #cc3b0f !important;
+}
+
 .tabitem-iframe::after {
   content: "</>";
   padding-left: 0.33em;
diff --git a/src/res/css/common.scss b/src/res/css/common.scss
index 3576c81..048a170 100644
--- a/src/res/css/common.scss
+++ b/src/res/css/common.scss
@@ -255,6 +255,10 @@ small {
 
 /** misc **/
 
+.warning-color {
+  color: #d6ba4a;
+}
+
 .warning {
   color: #d6ba4a;
   padding-left: 35px;