update: removed single click-through option

since vanish does the same there's no need to have it available and it can lead to confusion
Enubia · Jan 1, 2024 · cbba964 · cbba964
1 parent 9dd1563
commit cbba964
Show file tree

Hide file tree

Showing 6 changed files with 19 additions and 19 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -12,7 +12,7 @@
 	],
 	"eslint.enable": true,
 	"editor.codeActionsOnSave": {
-		"source.fixAll.eslint": true
+		"source.fixAll.eslint": "explicit"
 	},
 	"search.exclude": {
 		".eslintcache": true

diff --git a/electron/main/trayIcon.ts b/electron/main/trayIcon.ts
@@ -62,16 +62,17 @@ export default class TrayIcon {
                     }
                 },
             },
-            {
-                label: 'Disable Click-Through',
-                type: 'normal',
-                click: () => {
-                    log.info('Disabling Click-Through');
-                    this.store.set('savedWindowState.isClickThrough', false);
+            // should not be needed anymore, the previous menu item should be enough
+            // {
+            //     label: 'Disable Click-Through',
+            //     type: 'normal',
+            //     click: () => {
+            //         log.info('Disabling Click-Through');
+            //         this.store.set('savedWindowState.isClickThrough', false);
 
-                    this.overlay.setIgnoreMouseEvents(false);
-                },
-            },
+            //         this.overlay.setIgnoreMouseEvents(false);
+            //     },
+            // },
             {
                 label: 'Exit',
                 click: () => {

diff --git a/src/components/header/Dropdown.vue b/src/components/header/Dropdown.vue
@@ -21,9 +21,9 @@
             <li v-if="!isSettingsOpen">
                 <a @click="showSettings">{{ t('header.dropdown.settings') }}</a>
             </li>
-            <li v-if="!isSettingsOpen">
+            <!-- <li v-if="!isSettingsOpen">
                 <a @click="setClickThrough">{{ t('header.dropdown.set-click-through') }}</a>
-            </li>
+            </li> -->
             <li>
                 <a @click="toggleTheme">{{ t('header.dropdown.toggle-color-theme') }}</a>
             </li>
@@ -94,10 +94,10 @@ const toggleTheme = () => {
     ipcRenderer.send(IpcEvent.Rerender, 'child');
 };
 
-const setClickThrough = () => {
-    document.querySelector('details')?.removeAttribute('open');
-    ipcRenderer.send(IpcEvent.SetClickThrough);
-};
+// const setClickThrough = () => {
+//     document.querySelector('details')?.removeAttribute('open');
+//     ipcRenderer.send(IpcEvent.SetClickThrough);
+// };
 
 const emitVanish = () => {
     document.querySelector('details')?.removeAttribute('open');

diff --git a/src/components/settings/KapChat.vue b/src/components/settings/KapChat.vue
@@ -1,3 +1,4 @@
+<!-- eslint-disable @stylistic/max-len -->
 <template>
     <div id="default-channel">
         <label for="default-channel-input">

diff --git a/src/pages/Settings.vue b/src/pages/Settings.vue
@@ -97,7 +97,7 @@ const showCSSEditor = ref(false);
 const showJSEditor = ref(false);
 const showUpdates = ref(false);
 
-const showView = <T = 'general' | 'options' | 'css' | 'js' | 'updates',>(view: T) => {
+const showView = <T = 'general' | 'options' | 'css' | 'js' | 'updates'>(view: T) => {
     switch (view) {
         case 'general':
             document.querySelector('#options')?.removeAttribute('active');

diff --git a/src/schema.d.ts b/src/schema.d.ts
@@ -1,5 +1,3 @@
-// eslint-disable-next-line max-len
-// eslint-disable-next-line unused-imports/no-unused-imports, @typescript-eslint/no-unused-vars
 import { DefineLocaleMessage } from 'vue-i18n';
 
 import { MessageSchema } from '../i18n/schema';