From c2cb82c6a8b18d02e9317e02d7955133741393c0 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 21:39:49 +0100 Subject: [PATCH 01/29] #48 Installed typescript dependencies --- package-lock.json | 44 ++++++++++++++++++++++++++++++++++++++++++++ package.json | 2 ++ 2 files changed, 46 insertions(+) diff --git a/package-lock.json b/package-lock.json index b4ec7a2..c0c1996 100644 --- a/package-lock.json +++ b/package-lock.json @@ -6736,6 +6736,44 @@ } } }, + "gulp-typescript": { + "version": "6.0.0-alpha.1", + "resolved": "https://registry.npmjs.org/gulp-typescript/-/gulp-typescript-6.0.0-alpha.1.tgz", + "integrity": "sha512-KoT0TTfjfT7w3JItHkgFH1T/zK4oXWC+a8xxKfniRfVcA0Fa1bKrIhztYelYmb+95RB80OLMBreknYkdwzdi2Q==", + "dev": true, + "requires": { + "ansi-colors": "^4.1.1", + "plugin-error": "^1.0.1", + "source-map": "^0.7.3", + "through2": "^3.0.1", + "vinyl": "^2.2.0", + "vinyl-fs": "^3.0.3" + }, + "dependencies": { + "ansi-colors": { + "version": "4.1.1", + "resolved": "https://registry.npmjs.org/ansi-colors/-/ansi-colors-4.1.1.tgz", + "integrity": "sha512-JoX0apGbHaUJBNl6yF+p6JAFYZ666/hhCGKN5t9QFjbJQKUU/g8MNbFDbvfrgKXvI1QpZplPOnwIo99lX/AAmA==", + "dev": true + }, + "source-map": { + "version": "0.7.3", + "resolved": "https://registry.npmjs.org/source-map/-/source-map-0.7.3.tgz", + "integrity": "sha512-CkCj6giN3S+n9qrYiBTX5gystlENnRW5jZeNLHpe6aue+SrHcG5VYwujhW9s4dY31mEGsxBDrHR6oI69fTXsaQ==", + "dev": true + }, + "through2": { + "version": "3.0.2", + "resolved": "https://registry.npmjs.org/through2/-/through2-3.0.2.tgz", + "integrity": "sha512-enaDQ4MUyP2W6ZyT6EsMzqBPZaM/avg8iuo+l2d3QCs0J+6RaqkHV/2/lOwDTueBHeJ/2LG9lrLW3d5rWPucuQ==", + "dev": true, + "requires": { + "inherits": "^2.0.4", + "readable-stream": "2 || 3" + } + } + } + }, "gulp-zip": { "version": "5.0.2", "resolved": "https://registry.npmjs.org/gulp-zip/-/gulp-zip-5.0.2.tgz", @@ -12597,6 +12635,12 @@ "integrity": "sha1-hnrHTjhkGHsdPUfZlqeOxciDB3c=", "dev": true }, + "typescript": { + "version": "4.1.3", + "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.1.3.tgz", + "integrity": "sha512-B3ZIOf1IKeH2ixgHhj6la6xdwR9QrLC5d1VKeCSY4tvkqhF2eqd9O7txNlS0PO3GrBAFIdr3L1ndNwteUbZLYg==", + "dev": true + }, "umd": { "version": "3.0.3", "resolved": "https://registry.npmjs.org/umd/-/umd-3.0.3.tgz", diff --git a/package.json b/package.json index b2dd2dc..a7b41f4 100644 --- a/package.json +++ b/package.json @@ -65,11 +65,13 @@ "gulp-size": "^3.0.0", "gulp-sourcemaps": "^3.0.0", "gulp-terser": "^2.0.0", + "gulp-typescript": "^6.0.0-alpha.1", "gulp-zip": "^5.0.2", "merge-stream": "^2.0.0", "postcss": "^8.2.1", "sass": "^1.30.0", "terser": "^5.5.1", + "typescript": "^4.1.3", "vinyl-buffer": "^1.0.1", "vinyl-source-stream": "^2.0.0" } From 79c2784d73b285970facb785e829adf0690b6625 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 21:43:06 +0100 Subject: [PATCH 02/29] #48 Changed file extensions from js to ts --- src/javascript/{app.js => app.ts} | 0 .../browserApi/{storageProvider.js => storageProvider.ts} | 0 .../configuration/{configuration.js => configuration.ts} | 0 src/javascript/enhancements/{anilyr.js => anilyr.ts} | 0 .../enhancements/{animeRequests.js => animeRequests.ts} | 0 .../enhancements/{cssEnhancements.js => cssEnhancements.ts} | 0 src/javascript/enhancements/{fontColor.js => fontColor.ts} | 0 .../enhancements/{languageDisplay.js => languageDisplay.ts} | 0 .../enhancements/{notifications.js => notifications.ts} | 0 src/javascript/enhancements/{quickSearch.js => quickSearch.ts} | 0 .../enhancements/{watch2getherChat.js => watch2getherChat.ts} | 0 src/javascript/{settings.js => settings.ts} | 0 src/javascript/utils/{aniwatchCore.js => aniwatchCore.ts} | 0 src/javascript/utils/{colors.js => colors.ts} | 0 src/javascript/utils/{helpers.js => helpers.ts} | 0 15 files changed, 0 insertions(+), 0 deletions(-) rename src/javascript/{app.js => app.ts} (100%) rename src/javascript/browserApi/{storageProvider.js => storageProvider.ts} (100%) rename src/javascript/configuration/{configuration.js => configuration.ts} (100%) rename src/javascript/enhancements/{anilyr.js => anilyr.ts} (100%) rename src/javascript/enhancements/{animeRequests.js => animeRequests.ts} (100%) rename src/javascript/enhancements/{cssEnhancements.js => cssEnhancements.ts} (100%) rename src/javascript/enhancements/{fontColor.js => fontColor.ts} (100%) rename src/javascript/enhancements/{languageDisplay.js => languageDisplay.ts} (100%) rename src/javascript/enhancements/{notifications.js => notifications.ts} (100%) rename src/javascript/enhancements/{quickSearch.js => quickSearch.ts} (100%) rename src/javascript/enhancements/{watch2getherChat.js => watch2getherChat.ts} (100%) rename src/javascript/{settings.js => settings.ts} (100%) rename src/javascript/utils/{aniwatchCore.js => aniwatchCore.ts} (100%) rename src/javascript/utils/{colors.js => colors.ts} (100%) rename src/javascript/utils/{helpers.js => helpers.ts} (100%) diff --git a/src/javascript/app.js b/src/javascript/app.ts similarity index 100% rename from src/javascript/app.js rename to src/javascript/app.ts diff --git a/src/javascript/browserApi/storageProvider.js b/src/javascript/browserApi/storageProvider.ts similarity index 100% rename from src/javascript/browserApi/storageProvider.js rename to src/javascript/browserApi/storageProvider.ts diff --git a/src/javascript/configuration/configuration.js b/src/javascript/configuration/configuration.ts similarity index 100% rename from src/javascript/configuration/configuration.js rename to src/javascript/configuration/configuration.ts diff --git a/src/javascript/enhancements/anilyr.js b/src/javascript/enhancements/anilyr.ts similarity index 100% rename from src/javascript/enhancements/anilyr.js rename to src/javascript/enhancements/anilyr.ts diff --git a/src/javascript/enhancements/animeRequests.js b/src/javascript/enhancements/animeRequests.ts similarity index 100% rename from src/javascript/enhancements/animeRequests.js rename to src/javascript/enhancements/animeRequests.ts diff --git a/src/javascript/enhancements/cssEnhancements.js b/src/javascript/enhancements/cssEnhancements.ts similarity index 100% rename from src/javascript/enhancements/cssEnhancements.js rename to src/javascript/enhancements/cssEnhancements.ts diff --git a/src/javascript/enhancements/fontColor.js b/src/javascript/enhancements/fontColor.ts similarity index 100% rename from src/javascript/enhancements/fontColor.js rename to src/javascript/enhancements/fontColor.ts diff --git a/src/javascript/enhancements/languageDisplay.js b/src/javascript/enhancements/languageDisplay.ts similarity index 100% rename from src/javascript/enhancements/languageDisplay.js rename to src/javascript/enhancements/languageDisplay.ts diff --git a/src/javascript/enhancements/notifications.js b/src/javascript/enhancements/notifications.ts similarity index 100% rename from src/javascript/enhancements/notifications.js rename to src/javascript/enhancements/notifications.ts diff --git a/src/javascript/enhancements/quickSearch.js b/src/javascript/enhancements/quickSearch.ts similarity index 100% rename from src/javascript/enhancements/quickSearch.js rename to src/javascript/enhancements/quickSearch.ts diff --git a/src/javascript/enhancements/watch2getherChat.js b/src/javascript/enhancements/watch2getherChat.ts similarity index 100% rename from src/javascript/enhancements/watch2getherChat.js rename to src/javascript/enhancements/watch2getherChat.ts diff --git a/src/javascript/settings.js b/src/javascript/settings.ts similarity index 100% rename from src/javascript/settings.js rename to src/javascript/settings.ts diff --git a/src/javascript/utils/aniwatchCore.js b/src/javascript/utils/aniwatchCore.ts similarity index 100% rename from src/javascript/utils/aniwatchCore.js rename to src/javascript/utils/aniwatchCore.ts diff --git a/src/javascript/utils/colors.js b/src/javascript/utils/colors.ts similarity index 100% rename from src/javascript/utils/colors.js rename to src/javascript/utils/colors.ts diff --git a/src/javascript/utils/helpers.js b/src/javascript/utils/helpers.ts similarity index 100% rename from src/javascript/utils/helpers.js rename to src/javascript/utils/helpers.ts From 598f456f1be38b83152abd7d7ecdf559e66b619d Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 21:46:45 +0100 Subject: [PATCH 03/29] #48 Prepared build scripts --- gulpfile.js | 6 +++-- package-lock.json | 57 +++++++++++++++++++++++++++++++++++++++++++++++ package.json | 1 + 3 files changed, 62 insertions(+), 2 deletions(-) diff --git a/gulpfile.js b/gulpfile.js index 11c9db9..e4fbce1 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -4,6 +4,7 @@ const gulpLoadPlugins = require('gulp-load-plugins') const terser = require('terser'); const del = require('del'); const browserify = require('browserify'); +const tsify = require('tsify') const babelify = require('babelify'); const source = require('vinyl-source-stream'); const buffer = require('vinyl-buffer'); @@ -124,13 +125,14 @@ gulp.task('scripts', () => { const streams = []; modules.forEach(module => { - inputs.push(`${src.scripts}/${module}.js`); - streams.push(source(`${module}.js`)); + inputs.push(`${src.scripts}/${module}.ts`); + streams.push(source(`${module}.ts`)); }); const b = browserify(inputs, { debug: isDev }); let outstream = b + .plugin(tsify) .transform('babelify') .plugin(factor, { outputs: streams }) .bundle() diff --git a/package-lock.json b/package-lock.json index c0c1996..497b45e 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1651,6 +1651,12 @@ "integrity": "sha1-qCJQ3bABXponyoLoLqYDu/pF768=", "dev": true }, + "any-promise": { + "version": "1.3.0", + "resolved": "https://registry.npmjs.org/any-promise/-/any-promise-1.3.0.tgz", + "integrity": "sha1-q8av7tzqUugJzcA3au0845Y10X8=", + "dev": true + }, "anymatch": { "version": "2.0.0", "resolved": "https://registry.npmjs.org/anymatch/-/anymatch-2.0.0.tgz", @@ -12171,6 +12177,12 @@ "get-stdin": "^4.0.1" } }, + "strip-json-comments": { + "version": "2.0.1", + "resolved": "https://registry.npmjs.org/strip-json-comments/-/strip-json-comments-2.0.1.tgz", + "integrity": "sha1-PFMZQukIwml8DsNEhYwobHygpgo=", + "dev": true + }, "strip-outer": { "version": "1.0.1", "resolved": "https://registry.npmjs.org/strip-outer/-/strip-outer-1.0.1.tgz", @@ -12602,6 +12614,51 @@ "glob": "^7.1.2" } }, + "tsconfig": { + "version": "5.0.3", + "resolved": "https://registry.npmjs.org/tsconfig/-/tsconfig-5.0.3.tgz", + "integrity": "sha1-X0J45wGACWeo/Dg/0ZZIh48qbjo=", + "dev": true, + "requires": { + "any-promise": "^1.3.0", + "parse-json": "^2.2.0", + "strip-bom": "^2.0.0", + "strip-json-comments": "^2.0.0" + }, + "dependencies": { + "parse-json": { + "version": "2.2.0", + "resolved": "https://registry.npmjs.org/parse-json/-/parse-json-2.2.0.tgz", + "integrity": "sha1-9ID0BDTvgHQfhGkJn43qGPVaTck=", + "dev": true, + "requires": { + "error-ex": "^1.2.0" + } + } + } + }, + "tsify": { + "version": "5.0.2", + "resolved": "https://registry.npmjs.org/tsify/-/tsify-5.0.2.tgz", + "integrity": "sha512-Pdo3ZO8CAgbQgNcFRBmfbgsPP+4TsD0itbSF5YgTnxKBXfg6WkQ79e4/bqBaq/7cEYa7vIOM1pHxnux8rJJnzg==", + "dev": true, + "requires": { + "convert-source-map": "^1.1.0", + "fs.realpath": "^1.0.0", + "object-assign": "^4.1.0", + "semver": "^6.1.0", + "through2": "^2.0.0", + "tsconfig": "^5.0.3" + }, + "dependencies": { + "semver": { + "version": "6.3.0", + "resolved": "https://registry.npmjs.org/semver/-/semver-6.3.0.tgz", + "integrity": "sha512-b39TBaTSfV6yBrapU89p5fKekE2m/NwnDocOVruQFS1/veMgdzuPcnOM34M6CwxW8jH/lxEa5rBoDeUwu5HHTw==", + "dev": true + } + } + }, "tty-browserify": { "version": "0.0.1", "resolved": "https://registry.npmjs.org/tty-browserify/-/tty-browserify-0.0.1.tgz", diff --git a/package.json b/package.json index a7b41f4..6ec0880 100644 --- a/package.json +++ b/package.json @@ -71,6 +71,7 @@ "postcss": "^8.2.1", "sass": "^1.30.0", "terser": "^5.5.1", + "tsify": "^5.0.2", "typescript": "^4.1.3", "vinyl-buffer": "^1.0.1", "vinyl-source-stream": "^2.0.0" From 43e6cfc8f31b4f917b14fa5daee75582d542f7a8 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:08:50 +0100 Subject: [PATCH 04/29] #48 First types --- package-lock.json | 37 ++++++++++++++++++++ package.json | 4 ++- src/javascript/browserApi/storageProvider.ts | 31 +++++++++------- tsconfig.json | 6 ++++ 4 files changed, 65 insertions(+), 13 deletions(-) create mode 100644 tsconfig.json diff --git a/package-lock.json b/package-lock.json index 497b45e..fff28cd 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1488,12 +1488,37 @@ "dev": true, "optional": true }, + "@types/chrome": { + "version": "0.0.127", + "resolved": "https://registry.npmjs.org/@types/chrome/-/chrome-0.0.127.tgz", + "integrity": "sha512-hBB9EApLYKKn2GvklVkTxVP6vZvxsH9okyIRUinNtMzZHIgIKWQk/ESbX+O5g4Bihfy38+aFGn7Kl7Cxou5JUg==", + "dev": true, + "requires": { + "@types/filesystem": "*", + "@types/har-format": "*" + } + }, "@types/color-name": { "version": "1.1.1", "resolved": "https://registry.npmjs.org/@types/color-name/-/color-name-1.1.1.tgz", "integrity": "sha512-rr+OQyAjxze7GgWrSaJwydHStIhHq2lvY3BOC2Mj7KnzI7XK0Uw1TOOdI9lDoajEbSWLiYgoo4f1R51erQfhPQ==", "dev": true }, + "@types/filesystem": { + "version": "0.0.29", + "resolved": "https://registry.npmjs.org/@types/filesystem/-/filesystem-0.0.29.tgz", + "integrity": "sha512-85/1KfRedmfPGsbK8YzeaQUyV1FQAvMPMTuWFQ5EkLd2w7szhNO96bk3Rh/SKmOfd9co2rCLf0Voy4o7ECBOvw==", + "dev": true, + "requires": { + "@types/filewriter": "*" + } + }, + "@types/filewriter": { + "version": "0.0.28", + "resolved": "https://registry.npmjs.org/@types/filewriter/-/filewriter-0.0.28.tgz", + "integrity": "sha1-wFTor02d11205jq8dviFFocU1LM=", + "dev": true + }, "@types/glob": { "version": "7.1.3", "resolved": "https://registry.npmjs.org/@types/glob/-/glob-7.1.3.tgz", @@ -1504,6 +1529,12 @@ "@types/node": "*" } }, + "@types/har-format": { + "version": "1.2.5", + "resolved": "https://registry.npmjs.org/@types/har-format/-/har-format-1.2.5.tgz", + "integrity": "sha512-IG8AE1m2pWtPqQ7wXhFhy6Q59bwwnLwO36v5Rit2FrbXCIp8Sk8E2PfUCreyrdo17STwFSKDAkitVuVYbpEHvQ==", + "dev": true + }, "@types/minimatch": { "version": "3.0.3", "resolved": "https://registry.npmjs.org/@types/minimatch/-/minimatch-3.0.3.tgz", @@ -13115,6 +13146,12 @@ "integrity": "sha512-2ham8XPWTONajOR0ohOKOHXkm3+gaBmGut3SRuu75xLd/RRaY6vqgh8NBYYk7+RW3u5AtzPQZG8F10LHkl0lAQ==", "dev": true }, + "web-ext-types": { + "version": "3.2.1", + "resolved": "https://registry.npmjs.org/web-ext-types/-/web-ext-types-3.2.1.tgz", + "integrity": "sha512-oQZYDU3W8X867h8Jmt3129kRVKklz70db40Y6OzoTTuzOJpF/dB2KULJUf0txVPyUUXuyzV8GmT3nVvRHoG+Ew==", + "dev": true + }, "which": { "version": "1.3.1", "resolved": "https://registry.npmjs.org/which/-/which-1.3.1.tgz", diff --git a/package.json b/package.json index 6ec0880..d32c98f 100644 --- a/package.json +++ b/package.json @@ -44,6 +44,7 @@ "@babel/plugin-proposal-private-methods": "^7.12.1", "@babel/preset-env": "^7.12.11", "@babel/register": "^7.12.10", + "@types/chrome": "0.0.127", "babelify": "^10.0.0", "browserify": "^17.0.0", "cross-env": "^7.0.3", @@ -74,6 +75,7 @@ "tsify": "^5.0.2", "typescript": "^4.1.3", "vinyl-buffer": "^1.0.1", - "vinyl-source-stream": "^2.0.0" + "vinyl-source-stream": "^2.0.0", + "web-ext-types": "^3.2.1" } } diff --git a/src/javascript/browserApi/storageProvider.ts b/src/javascript/browserApi/storageProvider.ts index 0f84b1d..ff53fa8 100644 --- a/src/javascript/browserApi/storageProvider.ts +++ b/src/javascript/browserApi/storageProvider.ts @@ -1,15 +1,20 @@ const { assigned } = require("../utils/helpers") -class StorageProviderChromium { +export interface ICustomBrowserStorageProvider { + setData(key: string, value: string): void; + getData(key: string, defaultValue: string, callback: (x: string) => void): void; +} - setData(key, value) { +class StorageProviderChromium implements ICustomBrowserStorageProvider { + + setData(key: string, value: string): void { let obj = {}; obj[key] = value; this.getStorage().set(obj); } - getData(key, defaultValue, callback) { + getData(key: string, defaultValue: string, callback: (x: string) => void): void { this.getStorage().get(key, items => { if (assigned(items) && assigned(items[key])) { callback(items[key]); @@ -20,7 +25,7 @@ class StorageProviderChromium { }) } - getStorage() { + private getStorage(): chrome.storage.StorageArea { if (assigned(chrome.storage.sync)) { return chrome.storage.sync; } @@ -30,21 +35,21 @@ class StorageProviderChromium { } -class StorageProviderFirefox { +class StorageProviderFirefox implements ICustomBrowserStorageProvider { - setData(key, value) { + setData(key: string, value: string): void { let obj = {}; obj[key] = value; this.getStorage().set(obj); } - getData(key, defaultValue, callback) { + getData(key: string, defaultValue: string, callback: (x: string) => void): void { let promise = this.getStorage().get(key); promise.then(items => { if (assigned(items) && assigned(items[key])) { - callback(items[key]); + callback(items[key] as string); } else { callback(defaultValue); @@ -52,7 +57,7 @@ class StorageProviderFirefox { }); } - getStorage() { + private getStorage(): browser.storage.StorageArea { if (assigned(browser.storage.sync)) { return browser.storage.sync; } @@ -61,11 +66,13 @@ class StorageProviderFirefox { } } -let __storageProvieder; +let __storageProvieder: ICustomBrowserStorageProvider; function createStorageProvider() { // chrome based browser - if (assigned(chrome?.app)) { + // TODO: chrome.app? + // if (assigned(chrome?.app)) { + if (true) { __storageProvieder = new StorageProviderChromium(); } // firefox @@ -75,7 +82,7 @@ function createStorageProvider() { } -export function getGlobalStorageProvider() { +export function getGlobalStorageProvider(): ICustomBrowserStorageProvider { if (!assigned(__storageProvieder)) { createStorageProvider(); } diff --git a/tsconfig.json b/tsconfig.json new file mode 100644 index 0000000..05d6614 --- /dev/null +++ b/tsconfig.json @@ -0,0 +1,6 @@ +{ + "compilerOptions": { + // You have to explicitly set @types to get DefinitelyTyped type definitions + "typeRoots": ["node_modules/@types", "node_modules/web-ext-types"] + } + } \ No newline at end of file From d1e7df0264c80353fb4e9ff503f156881f2bb41e Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:15:28 +0100 Subject: [PATCH 05/29] #48 Typescript implementation for settings and configuration --- src/javascript/browserApi/storageProvider.ts | 16 +++++++++------- src/javascript/configuration/configuration.ts | 11 +++++++---- src/javascript/settings.ts | 4 ++-- 3 files changed, 18 insertions(+), 13 deletions(-) diff --git a/src/javascript/browserApi/storageProvider.ts b/src/javascript/browserApi/storageProvider.ts index ff53fa8..31a76bf 100644 --- a/src/javascript/browserApi/storageProvider.ts +++ b/src/javascript/browserApi/storageProvider.ts @@ -1,20 +1,22 @@ const { assigned } = require("../utils/helpers") +export type ConfigurationStorageBooleanCallback = (value: boolean) => void; + export interface ICustomBrowserStorageProvider { - setData(key: string, value: string): void; - getData(key: string, defaultValue: string, callback: (x: string) => void): void; + setDataAsBoolean(key: string, value: boolean): void; + getDataAsBoolean(key: string, defaultValue: boolean, callback: ConfigurationStorageBooleanCallback): void; } class StorageProviderChromium implements ICustomBrowserStorageProvider { - setData(key: string, value: string): void { + setDataAsBoolean(key: string, value: boolean): void { let obj = {}; obj[key] = value; this.getStorage().set(obj); } - getData(key: string, defaultValue: string, callback: (x: string) => void): void { + getDataAsBoolean(key: string, defaultValue: boolean, callback: ConfigurationStorageBooleanCallback): void { this.getStorage().get(key, items => { if (assigned(items) && assigned(items[key])) { callback(items[key]); @@ -37,19 +39,19 @@ class StorageProviderChromium implements ICustomBrowserStorageProvider { class StorageProviderFirefox implements ICustomBrowserStorageProvider { - setData(key: string, value: string): void { + setDataAsBoolean(key: string, value: boolean): void { let obj = {}; obj[key] = value; this.getStorage().set(obj); } - getData(key: string, defaultValue: string, callback: (x: string) => void): void { + getDataAsBoolean(key: string, defaultValue: boolean, callback: ConfigurationStorageBooleanCallback): void { let promise = this.getStorage().get(key); promise.then(items => { if (assigned(items) && assigned(items[key])) { - callback(items[key] as string); + callback(items[key] as boolean); } else { callback(defaultValue); diff --git a/src/javascript/configuration/configuration.ts b/src/javascript/configuration/configuration.ts index a0939f6..9f7136d 100644 --- a/src/javascript/configuration/configuration.ts +++ b/src/javascript/configuration/configuration.ts @@ -1,4 +1,4 @@ -import { getGlobalStorageProvider } from "../browserApi/storageProvider"; +import { ConfigurationStorageBooleanCallback, getGlobalStorageProvider } from "../browserApi/storageProvider"; import { assigned } from "../utils/helpers"; // website @@ -16,17 +16,20 @@ export const SETTINGS_playerAutoplayAfterScreenshot = 'playerAutoplayAfterScreen // w2g export const SETTINGS_w2gDisplayCharacterCounter = 'w2gDisplayCharacterCounter'; class Configuration { + settingsCache: Map; + constructor() { this.settingsCache = new Map(); } - getProperty(key, callback) { + getProperty(key: string, callback: ConfigurationStorageBooleanCallback): void { if (this.settingsCache.has(key)) { callback(this.settingsCache.get(key)); } else { + // TODO see commenz // OOOPS // currently all settings are default true. This isn´t a problem but there should be much better soloutions after migration to typescript.... - getGlobalStorageProvider().getData(key, true, value => { + getGlobalStorageProvider().getDataAsBoolean(key, true, value => { this.settingsCache.set(key, value); callback(value); }); @@ -34,7 +37,7 @@ class Configuration { } } -let __globalConfig; +let __globalConfig: Configuration; export function getGlobalConfiguration() { if (!assigned(__globalConfig)) { diff --git a/src/javascript/settings.ts b/src/javascript/settings.ts index 86e4208..db82c4e 100644 --- a/src/javascript/settings.ts +++ b/src/javascript/settings.ts @@ -5,7 +5,7 @@ const OPTION_SELECTOR = 'input[type="checkbox"'; function storeOptions() { document.querySelectorAll(OPTION_SELECTOR).forEach(optionElement => { - getGlobalStorageProvider().setData(optionElement.id, optionElement.checked); + getGlobalStorageProvider().setDataAsBoolean(optionElement.id, optionElement.checked); }); } @@ -13,7 +13,7 @@ function restoreOptions() { document.querySelectorAll(OPTION_SELECTOR).forEach(optionElement => { let defaultValue = optionElement.dataset.defaultValue === 'true' ? true : false; - getGlobalStorageProvider().getData(optionElement.id, defaultValue, value => { + getGlobalStorageProvider().getDataAsBoolean(optionElement.id, defaultValue, value => { optionElement.checked = value; }); }); From ec87c72542ba039e84f7e74d97aa7e2f01667f14 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:25:46 +0100 Subject: [PATCH 06/29] #48 Added types to core --- src/javascript/utils/aniwatchCore.ts | 47 ++++++++++++++++++---------- 1 file changed, 30 insertions(+), 17 deletions(-) diff --git a/src/javascript/utils/aniwatchCore.ts b/src/javascript/utils/aniwatchCore.ts index 0900f8d..28eabc1 100644 --- a/src/javascript/utils/aniwatchCore.ts +++ b/src/javascript/utils/aniwatchCore.ts @@ -1,11 +1,24 @@ import * as helper from './helpers'; -/* SCRIPT LOGICS */ -let __scripts = []; -let __afterLoadScripts = []; -let __afterLocationChangeScripts = []; +type ScriptCallback = () => void; +type NodeScriptCallback = (node: Node) => void; -export function initCore() { +type ScriptObj = { + function: ScriptCallback, + pattern: string +} + +type NodeScriptObj = { + function: NodeScriptCallback, + pattern: string +} + +/* SCRIPT LOGICS */ +let __scripts: Array = []; +let __afterLoadScripts: Array = []; +let __afterLocationChangeScripts: Array = []; + +export function initCore(): void { let observer = new MutationObserver(mutations => { mutations.forEach(mutation => { for (let i = 0; i < mutation.addedNodes.length; i++) { @@ -46,11 +59,11 @@ export function initCore() { helper.onReady(() => awaitPageLoaded()); } -export function registerScript(func, pattern = '.*') { - __scripts.push({ "function": func, "pattern": pattern }); +export function registerScript(func: NodeScriptCallback, pattern: string = '.*'): void { + __scripts.push({ function: func, pattern: pattern }); } -export function runScripts(node) { +export function runScripts(node: Node): void { __scripts.forEach(script => { if (window.location.pathname.match(script.pattern)) { script.function(node); @@ -58,20 +71,20 @@ export function runScripts(node) { }); } -function findPreloader() { +function findPreloader(): HTMLElement { return document.getElementById('preloader'); } -export function runAfterLoad(func, pattern = '.*') { +export function runAfterLoad(func: ScriptCallback, pattern: string = '.*'): void { let preloader = findPreloader(); if (typeof preloader !== undefined && preloader.style.display !== "none") { - __afterLoadScripts.push({ "function": func, "pattern": pattern }); + __afterLoadScripts.push({ function: func, pattern: pattern }); } else { func(); } } -function awaitPageLoaded() { +function awaitPageLoaded(): void { let preLoader = findPreloader(); let runScripts = () => { @@ -87,9 +100,9 @@ function awaitPageLoaded() { return; } - let loop = setInterval(() => { + let loop = window.setInterval(() => { if (preLoader.style.display === "none" && document.readyState === 'complete') { - clearInterval(loop); + window.clearInterval(loop); runScripts(); } @@ -97,12 +110,12 @@ function awaitPageLoaded() { } /* PATHNAME LOGIC */ -export function runAfterLocationChange(func, pattern = '.*') { - __afterLocationChangeScripts.push({ "function": func, "pattern": pattern }); +export function runAfterLocationChange(func: ScriptCallback, pattern: string = '.*'): void { + __afterLocationChangeScripts.push({ function: func, pattern: pattern }); } /* LOGIN LOGIC */ -export function isLoggedIn() { +export function isLoggedIn(): boolean { let menu = document.getElementById('materialize-menu-dropdown'); let result = true; From 2059a5d427d43a29fb4e6e33fb4b92e466d14f67 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:27:38 +0100 Subject: [PATCH 07/29] #48 Added types to helper --- src/javascript/utils/helpers.ts | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/javascript/utils/helpers.ts b/src/javascript/utils/helpers.ts index 053aa18..12dd315 100644 --- a/src/javascript/utils/helpers.ts +++ b/src/javascript/utils/helpers.ts @@ -1,16 +1,16 @@ -export var isShiftPressed = false; -export var isCtrlPressed = false; +export var isShiftPressed: boolean = false; +export var isCtrlPressed: boolean = false; -export function isHtmlElement(object) { +export function isHtmlElement(object: any) { return object instanceof HTMLElement; } -export function initHelpers() { +export function initHelpers(): void { document.addEventListener('keydown', event => handleKeyDown(event)); document.addEventListener('keyup', event => handleKeyUp(event)); } -export function onReady(fn) { +export function onReady(fn: () => void) { if (document.readyState != 'loading') { fn(); } else { @@ -18,19 +18,19 @@ export function onReady(fn) { } } -export function assigned(obj) { +export function assigned(obj: any): boolean { return !(typeof obj === 'undefined' || obj === null); } -function handleKeyDown(event) { +function handleKeyDown(event: KeyboardEvent) { handleKeyToggle(event, true); } -function handleKeyUp(event) { +function handleKeyUp(event: KeyboardEvent) { handleKeyToggle(event, false); } -function handleKeyToggle(event, isPressed) { +function handleKeyToggle(event: KeyboardEvent, isPressed: boolean) { if (event.key === 'Shift') { isShiftPressed = isPressed; } else if (event.key === 'Control') { From 30bd7c675d2c52e5100e8b56f601bd8d6db55102 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:32:11 +0100 Subject: [PATCH 08/29] #48 Added types to anilyr enhancements --- src/javascript/enhancements/anilyr.ts | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) diff --git a/src/javascript/enhancements/anilyr.ts b/src/javascript/enhancements/anilyr.ts index 2b4d6cb..98c6db3 100644 --- a/src/javascript/enhancements/anilyr.ts +++ b/src/javascript/enhancements/anilyr.ts @@ -5,26 +5,27 @@ import * as helper from '../utils/helpers'; const SCREENSHOT_TOOLTIP_ID = 'anilyr-screenshots-tooltip'; const PLAYER_ID = 'player'; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_playerAutoplayAfterScreenshot, value => { if (value) { core.registerScript(node => { - if (helper.isHtmlElement(node) && node.id === SCREENSHOT_TOOLTIP_ID) { - observeScreenshotTooltip(node); + let element = node as HTMLElement; + if (helper.assigned(element) && element.id === SCREENSHOT_TOOLTIP_ID) { + observeScreenshotTooltip(element); } }, "^/anime/[0-9]*/[0-9]*$"); } }); } -function observeScreenshotTooltip(tooltip) { +function observeScreenshotTooltip(tooltip: HTMLElement): void { let observer = new MutationObserver(mutations => { mutations.forEach(mutation => { // Switched to invisible - if (!mutation.oldValue.includes('display: none') && mutation.target.style.display == 'none') { - let player = findPlayer(); - if (helper.assigned(player)) { - resumePlayer(player); + if (!mutation.oldValue.includes('display: none') && helper.isHtmlElement(mutation.target) && (mutation.target as HTMLElement).style.display == 'none') { + let playerElement = findPlayerElement(); + if (helper.assigned(playerElement)) { + resumePlayer(playerElement); } } }); @@ -37,17 +38,15 @@ function observeScreenshotTooltip(tooltip) { }); } -function findPlayer() { - const PLAYER_TAG_NAME = 'VIDEO'; // tagName gives UpperCase - +function findPlayerElement(): HTMLVideoElement { let playerCandidate = document.getElementById(PLAYER_ID); - if (playerCandidate.tagName === PLAYER_TAG_NAME) { + if (playerCandidate instanceof HTMLVideoElement) { return playerCandidate; } return undefined; } -function resumePlayer(player) { +function resumePlayer(player: HTMLVideoElement) { player.play(); } \ No newline at end of file From f99034ceb3ea2aaa33c0358600f7bcb14d9d6002 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:40:15 +0100 Subject: [PATCH 09/29] #48 Added types for anime requests enhancements --- src/javascript/enhancements/animeRequests.ts | 33 ++++++++++---------- 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/src/javascript/enhancements/animeRequests.ts b/src/javascript/enhancements/animeRequests.ts index ce86be0..5a970dd 100644 --- a/src/javascript/enhancements/animeRequests.ts +++ b/src/javascript/enhancements/animeRequests.ts @@ -1,14 +1,13 @@ import { getGlobalConfiguration, SETTINGS_requestBeautifyPage } from '../configuration/configuration'; import * as core from '../utils/aniwatchCore'; import * as color from '../utils/colors'; -import * as helper from '../utils/helpers'; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_requestBeautifyPage, value => { if (value) { core.registerScript(node => { // run the scripts - if (helper.isHtmlElement(node)) { + if (node instanceof HTMLElement) { changeFollowedStarColor(node); changeBorderColorOwnRequests(node); removeUnknownUsers(node); @@ -18,20 +17,20 @@ export function init() { }); } -function changeFollowedStarColor(node) { - const starIcon = 'star'; +function changeFollowedStarColor(node: HTMLElement): void { + const STAR_ICON = 'star'; // find stars - let followedItems = Array.from(node.querySelectorAll('i')).filter(i => i.innerText.trim() === starIcon); + let followedItems = Array.from(node.querySelectorAll('i')).filter(i => i.innerText.trim() === STAR_ICON); // change color followedItems.forEach(item => item.style.color = color.aniBlue); } -function changeBorderColorOwnRequests(node) { - const targetTagName = 'MD-LIST-ITEM'; // tagName is upper case +function changeBorderColorOwnRequests(node: HTMLElement): void { + const TARGET_TAG_NAME = 'MD-LIST-ITEM'; // tagName is upper case - let updateFunc = item => { + let updateFunc = (item: HTMLElement): void => { let profileLink = item.querySelectorAll('a[href*="/profile/"]:not([href="/profile/false"])'); // highlight left border for own request @@ -41,7 +40,7 @@ function changeBorderColorOwnRequests(node) { } // are we target tag? - if (node.tagName === targetTagName) { + if (node.tagName === TARGET_TAG_NAME) { updateFunc(node); } else { // find items -> all @@ -49,15 +48,17 @@ function changeBorderColorOwnRequests(node) { // update borders requestItems.forEach(item => { - updateFunc(item); + if (item instanceof HTMLElement) { + updateFunc(item); + } }); } } -function removeUnknownUsers(node) { - const targetTagName = 'MD-LIST-ITEM'; // tagName is upper case +function removeUnknownUsers(node: HTMLElement): void { + const TARGET_TAG_NAME = 'MD-LIST-ITEM'; // tagName is upper case - let updateFunc = item => { + let updateFunc = (item: Element) => { // find user profile link -> own request let profileLink = item.querySelectorAll('a[href*="/profile/"]:not([href="/profile/false"])'); @@ -66,7 +67,7 @@ function removeUnknownUsers(node) { let lowerDiv = upperDiv.parentElement.nextElementSibling; // remember Data - let anime = lowerDiv.innerText; + let anime = lowerDiv.textContent; let profileData = upperDiv.innerHTML; // add user note if own request @@ -92,7 +93,7 @@ function removeUnknownUsers(node) { upperDiv.appendChild(bElement); } - if (node.tagName === targetTagName) { + if (node.tagName === TARGET_TAG_NAME) { updateFunc(node); } else { // find items -> all From 8a0f4a3db2c0a888fa099c1bdce5ef5ae3ffc732 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:46:28 +0100 Subject: [PATCH 10/29] #48 Added types for css enhancements --- .../enhancements/cssEnhancements.ts | 50 +++++++++---------- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/src/javascript/enhancements/cssEnhancements.ts b/src/javascript/enhancements/cssEnhancements.ts index ce62948..a7bd673 100644 --- a/src/javascript/enhancements/cssEnhancements.ts +++ b/src/javascript/enhancements/cssEnhancements.ts @@ -2,27 +2,27 @@ import { getGlobalConfiguration, SETTINGS_websiteHideUnusedTabs, SETTINGS_websit import * as core from '../utils/aniwatchCore'; import * as helper from '../utils/helpers'; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_websiteHideUnusedTabs, value => { // if disabled, add class to avoid our css optimizations if (!value) { - let disableFunc = node => { - if (helper.isHtmlElement(node)) { - let disableNode = node => { - node.classList.add('awp-hide-unused-disabled') - } + let disableFunc = (node: Element) => { + let disableNode = (node: Element) => { + node.classList.add('awp-hide-unused-disabled') + } - if (node.tagName === 'MD-TAB-ITEM') { - disableNode(node); - } - else { - node.querySelectorAll('md-tab-item').forEach(node => disableNode(node)); - } + if (node.tagName === 'MD-TAB-ITEM') { + disableNode(node); + } + else { + node.querySelectorAll('md-tab-item').forEach(node => disableNode(node)); } }; core.registerScript(node => { - disableFunc(node); + if (node instanceof Element) { + disableFunc(node); + } }, ".*"); core.runAfterLoad(() => { @@ -34,23 +34,23 @@ export function init() { getGlobalConfiguration().getProperty(SETTINGS_websiteOptimizeListAppearance, value => { // if disabled, add class to avoid our css optimizations if (!value) { - let disableFunc = node => { - if (helper.isHtmlElement(node)) { - let disableNode = node => { - node.classList.add('awp-list-disabled') - } + let disableFunc = (node: Element) => { + let disableNode = (node: Element) => { + node.classList.add('awp-list-disabled') + } - if (node.tagName === 'MD-LIST-ITEM') { - disableNode(node); - } - else { - node.querySelectorAll('md-list-item').forEach(node => disableNode(node)); - } + if (node.tagName === 'MD-LIST-ITEM') { + disableNode(node); + } + else { + node.querySelectorAll('md-list-item').forEach(node => disableNode(node)); } } core.registerScript(node => { - disableFunc(node); + if (node instanceof Element) { + disableFunc(node); + } }, ".*"); core.runAfterLoad(() => { From a8e5cf7e4b5fdabe4e1a8bd8a6feb51c19b54796 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:46:47 +0100 Subject: [PATCH 11/29] #48 Code optimization --- src/javascript/enhancements/cssEnhancements.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/src/javascript/enhancements/cssEnhancements.ts b/src/javascript/enhancements/cssEnhancements.ts index a7bd673..e30c1fc 100644 --- a/src/javascript/enhancements/cssEnhancements.ts +++ b/src/javascript/enhancements/cssEnhancements.ts @@ -1,6 +1,5 @@ import { getGlobalConfiguration, SETTINGS_websiteHideUnusedTabs, SETTINGS_websiteOptimizeListAppearance } from '../configuration/configuration'; import * as core from '../utils/aniwatchCore'; -import * as helper from '../utils/helpers'; export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_websiteHideUnusedTabs, value => { From b7c647e114c58da60b4c6510477d8b96a373bb99 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:55:09 +0100 Subject: [PATCH 12/29] #48 Added types for font color enhancements --- src/javascript/enhancements/fontColor.ts | 47 +++++++++++++----------- 1 file changed, 25 insertions(+), 22 deletions(-) diff --git a/src/javascript/enhancements/fontColor.ts b/src/javascript/enhancements/fontColor.ts index 6d3ddeb..59da798 100644 --- a/src/javascript/enhancements/fontColor.ts +++ b/src/javascript/enhancements/fontColor.ts @@ -7,7 +7,7 @@ const BADGE_CLASS = 'label'; const DARKCOLOR_CLASS = 'awp-fontColor-dark'; const __observedBadges = []; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_websiteOptimizeFontColors, value => { if (value) { core.runAfterLoad(() => { @@ -19,29 +19,29 @@ export function init() { }, ".*"); core.registerScript(node => { - checkRunColorOptimization(node); + if (node instanceof Element) { + checkRunColorOptimization(node); + } }, ".*"); } }); } -function checkRunColorOptimization(node) { +function checkRunColorOptimization(node: Element): void { // run the scripts - if (helper.isHtmlElement(node)) { - if (node.classList.contains(BADGE_CLASS)) { - tryRegisterObserverForBadge(node); - optimizeFontColorsBadges(node); - } - else { - node.querySelectorAll(`.${BADGE_CLASS}`).forEach(element => { - tryRegisterObserverForBadge(element); - optimizeFontColorsBadges(element); - }); - } + if (node.classList.contains(BADGE_CLASS)) { + tryRegisterObserverForBadge(node); + optimizeFontColorsBadges(node); + } + else { + node.querySelectorAll(`.${BADGE_CLASS}`).forEach(element => { + tryRegisterObserverForBadge(element); + optimizeFontColorsBadges(element); + }); } } -function tryRegisterObserverForBadge(badge) { +function tryRegisterObserverForBadge(badge: Node): void { // some badges change there color via late loading so we also have to observe the classlist // example: Navigating from a list to an anime -> "Currently Airing" late loads the color badge // this only happens when navigating from a list, direct loading works @@ -52,12 +52,15 @@ function tryRegisterObserverForBadge(badge) { let obsever = new MutationObserver(mutations => { mutations.forEach(mutation => { - // prevent recursive calls when our class is added / removed - if ((mutation.oldValue?.indexOf(DARKCOLOR_CLASS) ?? -1 ^ mutation.target?.classList?.indexOf(DARKCOLOR_CLASS) ?? -1) === 0) { - return; - } - else { - optimizeFontColorsBadges(mutation.target); + // prevent recursive calls when our class is added / removed + if (mutation.target instanceof Element) { + // TODO: Validate if that still works + if ((mutation.oldValue?.indexOf(DARKCOLOR_CLASS) ?? -1 ^ (mutation.target?.classList?.contains(DARKCOLOR_CLASS) ?? false ? 0 : -1)) === 0) { + return; + } + else { + optimizeFontColorsBadges(mutation.target); + } } }); }); @@ -71,7 +74,7 @@ function tryRegisterObserverForBadge(badge) { __observedBadges.push(badge); } -function optimizeFontColorsBadges(badge) { +function optimizeFontColorsBadges(badge: Element): void { let colorStr = window.getComputedStyle(badge, null).getPropertyValue('background-color'); // some elements do not have a computed background color From ecebbddddab40d50005a046b0f6289a72ad525eb Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 22:57:43 +0100 Subject: [PATCH 13/29] #48 Small improvment on already altered enhancements --- src/javascript/enhancements/anilyr.ts | 2 +- src/javascript/enhancements/animeRequests.ts | 2 +- src/javascript/enhancements/cssEnhancements.ts | 2 +- src/javascript/enhancements/fontColor.ts | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/javascript/enhancements/anilyr.ts b/src/javascript/enhancements/anilyr.ts index 98c6db3..7e32362 100644 --- a/src/javascript/enhancements/anilyr.ts +++ b/src/javascript/enhancements/anilyr.ts @@ -8,7 +8,7 @@ const PLAYER_ID = 'player'; export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_playerAutoplayAfterScreenshot, value => { if (value) { - core.registerScript(node => { + core.registerScript((node: Node) => { let element = node as HTMLElement; if (helper.assigned(element) && element.id === SCREENSHOT_TOOLTIP_ID) { observeScreenshotTooltip(element); diff --git a/src/javascript/enhancements/animeRequests.ts b/src/javascript/enhancements/animeRequests.ts index 5a970dd..9e9922c 100644 --- a/src/javascript/enhancements/animeRequests.ts +++ b/src/javascript/enhancements/animeRequests.ts @@ -5,7 +5,7 @@ import * as color from '../utils/colors'; export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_requestBeautifyPage, value => { if (value) { - core.registerScript(node => { + core.registerScript((node: Node) => { // run the scripts if (node instanceof HTMLElement) { changeFollowedStarColor(node); diff --git a/src/javascript/enhancements/cssEnhancements.ts b/src/javascript/enhancements/cssEnhancements.ts index e30c1fc..801e95f 100644 --- a/src/javascript/enhancements/cssEnhancements.ts +++ b/src/javascript/enhancements/cssEnhancements.ts @@ -18,7 +18,7 @@ export function init(): void { } }; - core.registerScript(node => { + core.registerScript((node: Node) => { if (node instanceof Element) { disableFunc(node); } diff --git a/src/javascript/enhancements/fontColor.ts b/src/javascript/enhancements/fontColor.ts index 59da798..f01435b 100644 --- a/src/javascript/enhancements/fontColor.ts +++ b/src/javascript/enhancements/fontColor.ts @@ -18,7 +18,7 @@ export function init(): void { checkRunColorOptimization(document.documentElement); }, ".*"); - core.registerScript(node => { + core.registerScript((node: Node) => { if (node instanceof Element) { checkRunColorOptimization(node); } From 413f35f35d84baafbf51c5eebe1f4c0594a99d43 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 23:08:04 +0100 Subject: [PATCH 14/29] #48 Added types for language display enhancements --- .../enhancements/languageDisplay.ts | 91 ++++++++++--------- 1 file changed, 49 insertions(+), 42 deletions(-) diff --git a/src/javascript/enhancements/languageDisplay.ts b/src/javascript/enhancements/languageDisplay.ts index a76529a..60f29a2 100644 --- a/src/javascript/enhancements/languageDisplay.ts +++ b/src/javascript/enhancements/languageDisplay.ts @@ -2,12 +2,14 @@ import { getGlobalConfiguration, SETTINGS_animeLanguageDisplay } from '../config import * as core from '../utils/aniwatchCore'; import * as helper from '../utils/helpers'; -export function init() { +const MANIPULATED_ATTR_NAME = 'awpManipulated'; + +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_animeLanguageDisplay, value => { if (value) { - core.registerScript(node => { + core.registerScript((node: Node) => { // run the scripts - if (helper.isHtmlElement(node)) { + if (node instanceof Element) { updateLanguageDisplay(node) } }, "^/anime/[0-9]*$"); @@ -15,34 +17,35 @@ export function init() { }); } -function updateLanguageDisplay(node) { - const listNodeName = 'MD-LIST-ITEM'; - const boxNodeName = 'DIV'; - const boxClassName = 'card-margin'; +function updateLanguageDisplay(node: Element): void { + const LIST_NODE_NAME = 'MD-LIST-ITEM'; + const BOX_NODE_NAME = 'DIV'; + const BOX_CLASS_NAME = 'card-margin'; - if (node.nodeName === listNodeName) { + if (node.nodeName === LIST_NODE_NAME) { updateLanguageDisplayListMode(node); } - else if (node.nodeName === boxNodeName && node.classList.contains(boxClassName)) { + else if (node.nodeName === BOX_NODE_NAME && node.classList.contains(BOX_CLASS_NAME)) { updateLanguageDisplayBoxMode(node); } } -function updateLanguageDisplayListMode(node) { +function updateLanguageDisplayListMode(node: Element): void { // last column with flags let col = node.querySelector('h3.layout-align-end-center'); - if (!helper.assigned(col) || col.awpManipulated) { + + if (!helper.assigned(col) || col.hasAttribute(MANIPULATED_ATTR_NAME)) { return; } doUpdateLanguageDisplay(col, false); } -function updateLanguageDisplayBoxMode(node) { +function updateLanguageDisplayBoxMode(node: Element): void { // last column with flags let col = node.querySelector('div.layout-align-end-start'); - if (!helper.assigned(col) || col.awpManipulated) { + if (!helper.assigned(col) || col.hasAttribute(MANIPULATED_ATTR_NAME)) { return; } @@ -50,27 +53,27 @@ function updateLanguageDisplayBoxMode(node) { } -function doUpdateLanguageDisplay(parent, isBoxedModed) { - const listLangPrefix = 'ep.lang.'; - const boxLangPrefix = 'episodeObject.lang.'; +function doUpdateLanguageDisplay(parent: Element, isBoxedModed: boolean): void { + const LIST_LANG_PREFIX = 'ep.lang.'; + const BOX_LANG_PREFIX = 'episodeObject.lang.'; // aniwatch uses different prefixes in list und box mode :/ - let realLangPrefix = isBoxedModed ? boxLangPrefix : listLangPrefix; + let realLangPrefix = isBoxedModed ? BOX_LANG_PREFIX : LIST_LANG_PREFIX; - const dubSuffix = 'dub'; - const subSuffix = 'sub'; + const DUB_SUFFIX = 'dub'; + const SUB_SUFFIX = 'sub'; - const dubIcon = 'volume_up'; - const subIcon = 'closed_caption'; - const zeroWidthSpace = ''; // ​ + const DUB_ICON = 'volume_up'; + const SUB_ICON = 'closed_caption'; + const ZERO_WIDTH_SPACE_CHARACTER = ''; // ​ - let subs = []; - let dubs = []; + let subs: Array = []; + let dubs: Array = []; // find subs let subCols = parent.querySelectorAll('[ng-hide*="sub"]'); - subCols.forEach(element => { + subCols.forEach((element: Element) => { let langAttr = element.attributes['ng-hide'].value; - let lang = langAttr.substring(langAttr.indexOf(realLangPrefix) + realLangPrefix.length, langAttr.indexOf(subSuffix)); + let lang = langAttr.substring(langAttr.indexOf(realLangPrefix) + realLangPrefix.length, langAttr.indexOf(SUB_SUFFIX)); if (element.attributes['aria-hidden'].value == 'false') { subs.push(lang); } @@ -78,9 +81,9 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { // find dubs let dubCols = parent.querySelectorAll('[ng-hide*="dub"]'); - dubCols.forEach(element => { + dubCols.forEach((element: Element) => { let langAttr = element.attributes['ng-hide'].value; - let lang = langAttr.substring(langAttr.indexOf(realLangPrefix) + realLangPrefix.length, langAttr.indexOf(dubSuffix)); + let lang = langAttr.substring(langAttr.indexOf(realLangPrefix) + realLangPrefix.length, langAttr.indexOf(DUB_SUFFIX)); if (element.attributes['aria-hidden'].value == 'false') { dubs.push(lang); } @@ -106,12 +109,12 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { let dubIconDiv = document.createElement('i'); if (iconsRequired) { dubIconDiv.classList.add('material-icons', 'mr-3'); - dubIconDiv.innerText = dubIcon; + dubIconDiv.innerText = DUB_ICON; } // add dummy with 24px for correct presentation else { dubIconDiv.style.height = '24px'; - dubIconDiv.innerText = zeroWidthSpace; + dubIconDiv.innerText = ZERO_WIDTH_SPACE_CHARACTER; } dubDiv.appendChild(dubIconDiv); @@ -132,16 +135,16 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { let subIconDiv = document.createElement('i'); if (iconsRequired) { subIconDiv.classList.add('material-icons', 'mr-3'); - subIconDiv.innerText = subIcon; + subIconDiv.innerText = SUB_ICON; } // add dummy with 24px for correct presentation else { subIconDiv.style.height = '24px'; - subIconDiv.innerText = zeroWidthSpace; + subIconDiv.innerText = ZERO_WIDTH_SPACE_CHARACTER; } subDiv.appendChild(subIconDiv); - subs.forEach(lang => { + subs.forEach((lang: string) => { let langIcon = document.createElement('i'); langIcon.classList.add('flag', `flag-${lang}`, 'mg-all-1'); subDiv.appendChild(langIcon); @@ -154,7 +157,7 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { } if (dubs.length > 0) { - dubs.forEach(lang => { + dubs.forEach((lang: string) => { let colDiv = document.createElement('div'); colDiv.setAttribute('layout', 'column'); colDiv.classList.add('layout-column'); @@ -167,12 +170,12 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { let dubIconDiv = document.createElement('i'); if (iconsRequired) { dubIconDiv.classList.add('material-icons', 'mr-3'); - dubIconDiv.innerText = dubIcon; + dubIconDiv.innerText = DUB_ICON; } // add dummy with 24px for correct presentation else { dubIconDiv.style.height = '24px'; - dubIconDiv.innerText = zeroWidthSpace; + dubIconDiv.innerText = ZERO_WIDTH_SPACE_CHARACTER; } dubDiv.appendChild(dubIconDiv); @@ -193,12 +196,12 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { let subIconDiv = document.createElement('i'); if (iconsRequired) { subIconDiv.classList.add('material-icons', 'mr-3'); - subIconDiv.innerText = subIcon; + subIconDiv.innerText = SUB_ICON; } // add dummy with 24px for correct presentation else { subIconDiv.style.height = '24px'; - subIconDiv.innerText = zeroWidthSpace; + subIconDiv.innerText = ZERO_WIDTH_SPACE_CHARACTER; } subDiv.appendChild(subIconDiv); @@ -216,13 +219,17 @@ function doUpdateLanguageDisplay(parent, isBoxedModed) { }); parent.querySelectorAll('.layout-column:not(:last-child)').forEach(div => { - div.style.borderRight = '1px solid rgba(155,155,155, 0.2)'; + if (div instanceof HTMLElement) { + div.style.borderRight = '1px solid rgba(155,155,155, 0.2)'; + } }) parent.querySelectorAll('.layout-column').forEach(div => { - div.style.paddingLeft = '2px'; - div.style.paddingRight = '2px'; + if (div instanceof HTMLElement) { + div.style.paddingLeft = '2px'; + div.style.paddingRight = '2px'; + } }) - parent.awpManipulated = true; + parent.setAttribute('awpManipulated', String(true)); } From 43b5fb7b1468bcee89271cbcfa216049137a41ef Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 23:10:56 +0100 Subject: [PATCH 15/29] #48 Added types for notification enhancements --- src/javascript/enhancements/notifications.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/javascript/enhancements/notifications.ts b/src/javascript/enhancements/notifications.ts index 10df5ff..768d419 100644 --- a/src/javascript/enhancements/notifications.ts +++ b/src/javascript/enhancements/notifications.ts @@ -2,7 +2,7 @@ import { getGlobalConfiguration, SETTINGS_websiteShowNotificationsCountInTab } f import * as core from '../utils/aniwatchCore'; import * as helper from '../utils/helpers'; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_websiteShowNotificationsCountInTab, value => { if (value) { core.runAfterLoad(() => { @@ -16,17 +16,17 @@ export function init() { }); } -function getNotificationCount() { +function getNotificationCount(): number { if (core.isLoggedIn()) { let menuUserText = document.getElementById('materialize-menu-dropdown').innerText.split('\n')[4]; - let notificationCount = menuUserText.match(/\d+/)?.[0] ?? 0; + let notificationCount = parseInt(menuUserText.match(/\d+/)?.[0]) ?? 0; return notificationCount; } else { return 0; } } -function updateNotificationsInTitle() { +function updateNotificationsInTitle(): void { let count = getNotificationCount(); if (helper.assigned(count) && count > 0) { From b0d0f66369afc17f448a11d1423d40cdfb1c96e3 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 23:19:18 +0100 Subject: [PATCH 16/29] #48 Added types for quick search enhancements --- src/javascript/enhancements/quickSearch.ts | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/javascript/enhancements/quickSearch.ts b/src/javascript/enhancements/quickSearch.ts index 4094756..95f52d0 100644 --- a/src/javascript/enhancements/quickSearch.ts +++ b/src/javascript/enhancements/quickSearch.ts @@ -5,7 +5,7 @@ import * as helper from '../utils/helpers'; const quickSearchID = 'ea-quickSearch'; const quickSearchLink = 'ea-quickSearchLink'; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_websiteDisplayQuickSearch, value => { if (value) { core.runAfterLoad(() => { @@ -15,7 +15,7 @@ export function init() { }); } -function initSearch() { +function initSearch(): void { let entry = document.createElement('li'); entry.setAttribute('ng-repeat', 'item in navbar'); entry.setAttribute('ng-class', '{\'anime-indicator\': item[\'@attributes\'].title==\'Anime\'}'); @@ -45,13 +45,13 @@ function initSearch() { document.addEventListener('keypress', event => handleSearchForShiftF(event)); // additionally, the last dropdown ul has a "right: 0px" style, which has to be fixed with auto, otherwhise it will pop up in the wrong position - Array.from(menu.querySelectorAll('ul.dropdown')).slice(-1)[0].style.right = 'auto'; + (Array.from(menu.querySelectorAll('ul.dropdown')).slice(-1)[0] as HTMLElement).style.right = 'auto'; } -function handleQuickSearch(event) { +function handleQuickSearch(event: KeyboardEvent): void { if (event.key === 'Enter') { - let quickSearchElement = document.getElementById(quickSearchID); - let linkElement = document.getElementById(quickSearchLink); + let quickSearchElement = document.getElementById(quickSearchID) as HTMLInputElement; + let linkElement = document.getElementById(quickSearchLink) as HTMLAnchorElement; let url = new URL(window.location.origin) url.pathname = '/search'; @@ -67,10 +67,10 @@ function handleQuickSearch(event) { } } -function handleSearchForShiftF(event) { +function handleSearchForShiftF(event: KeyboardEvent): void { if (helper.isShiftPressed) { // check if some kind of input is focused already; we then prevent our hotkey - if (document.activeElement instanceof HTMLInputElement || document.activeElement instanceof HTMLTextAreaElement || document.activeElement.isContentEditable) { + if (document.activeElement instanceof HTMLInputElement || document.activeElement instanceof HTMLTextAreaElement || ((document.activeElement as HTMLElement)?.isContentEditable ?? false)) { return; } From 36b12f539730d3c02efc0f4935e3376bc8005a0c Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 23:20:44 +0100 Subject: [PATCH 17/29] #48 Added types for watch2gether enhancements --- src/javascript/enhancements/watch2getherChat.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/javascript/enhancements/watch2getherChat.ts b/src/javascript/enhancements/watch2getherChat.ts index 0f102b3..8ca0526 100644 --- a/src/javascript/enhancements/watch2getherChat.ts +++ b/src/javascript/enhancements/watch2getherChat.ts @@ -3,7 +3,7 @@ import { v4 as uuidv4 } from 'uuid'; import { getGlobalConfiguration, SETTINGS_w2gDisplayCharacterCounter } from '../configuration/configuration'; import { assigned } from '../utils/helpers'; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_w2gDisplayCharacterCounter, value => { if (value) { core.runAfterLocationChange(() => { @@ -13,8 +13,8 @@ export function init() { }); } -function manipulateChatInput() { - let textarea = document.querySelector('.chat-input textarea'); +function manipulateChatInput(): void { + let textarea = document.querySelector('.chat-input textarea') as HTMLTextAreaElement; // avoid duplicate registration if (assigned(textarea.dataset.charCounterId)) { @@ -24,7 +24,7 @@ function manipulateChatInput() { addCharCounter(textarea); } -function addCharCounter(textarea) { +function addCharCounter(textarea: HTMLTextAreaElement): void { let chatDiv = textarea.parentElement.parentElement; // div with chat input and controls let controlRow = chatDiv.children[1]; // row with controls let btn = controlRow.querySelector('button'); // find send button @@ -45,7 +45,7 @@ function addCharCounter(textarea) { }); } -function updateCharCounter(textarea, charCounterSpan) { +function updateCharCounter(textarea: HTMLTextAreaElement, charCounterSpan: HTMLSpanElement): void { const SHAKE_CLASS = 'awp-w2g-chatCounter-max'; let current = textarea.value.length; From a743cc9897c6517d624fc3e8ea416af14fcb6745 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 23:22:47 +0100 Subject: [PATCH 18/29] #48 Added types for settings scripts --- src/javascript/settings.ts | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/src/javascript/settings.ts b/src/javascript/settings.ts index db82c4e..8d67105 100644 --- a/src/javascript/settings.ts +++ b/src/javascript/settings.ts @@ -5,25 +5,28 @@ const OPTION_SELECTOR = 'input[type="checkbox"'; function storeOptions() { document.querySelectorAll(OPTION_SELECTOR).forEach(optionElement => { - getGlobalStorageProvider().setDataAsBoolean(optionElement.id, optionElement.checked); + let optionInputElement = optionElement as HTMLInputElement; + getGlobalStorageProvider().setDataAsBoolean(optionInputElement.id, optionInputElement.checked); }); } function restoreOptions() { - document.querySelectorAll(OPTION_SELECTOR).forEach(optionElement => { - let defaultValue = optionElement.dataset.defaultValue === 'true' ? true : false; + document.querySelectorAll(OPTION_SELECTOR).forEach(optionElement => { + let optionInputElement = optionElement as HTMLInputElement; + let defaultValue = optionInputElement.dataset.defaultValue === 'true' ? true : false; - getGlobalStorageProvider().getDataAsBoolean(optionElement.id, defaultValue, value => { - optionElement.checked = value; + getGlobalStorageProvider().getDataAsBoolean(optionInputElement.id, defaultValue, value => { + optionInputElement.checked = value; }); }); } function resetOptions() { document.querySelectorAll(OPTION_SELECTOR).forEach(optionElement => { - let defaultValue = optionElement.dataset.defaultValue === 'true' ? true : false; + let optionInputElement = optionElement as HTMLInputElement; + let defaultValue = optionInputElement.dataset.defaultValue === 'true' ? true : false; - optionElement.checked = defaultValue; + optionInputElement.checked = defaultValue; }); } From f29de7f826d89b2516b1a96465f91d146123fdf1 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Mon, 28 Dec 2020 23:31:12 +0100 Subject: [PATCH 19/29] #48 Fixed minor error in the build script --- gulpfile.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gulpfile.js b/gulpfile.js index e4fbce1..0868042 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -126,7 +126,7 @@ gulp.task('scripts', () => { modules.forEach(module => { inputs.push(`${src.scripts}/${module}.ts`); - streams.push(source(`${module}.ts`)); + streams.push(source(`${module}.js`)); }); const b = browserify(inputs, { debug: isDev }); From 866d11a3d3f814230db8eb3ffa2308dc01923722 Mon Sep 17 00:00:00 2001 From: Daniel Date: Mon, 28 Dec 2020 23:40:12 +0100 Subject: [PATCH 20/29] Update src/javascript/configuration/configuration.ts #48 Fix Typo --- src/javascript/configuration/configuration.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/javascript/configuration/configuration.ts b/src/javascript/configuration/configuration.ts index 9f7136d..841301b 100644 --- a/src/javascript/configuration/configuration.ts +++ b/src/javascript/configuration/configuration.ts @@ -27,7 +27,7 @@ class Configuration { callback(this.settingsCache.get(key)); } else { - // TODO see commenz + // TODO see comment // OOOPS // currently all settings are default true. This isn´t a problem but there should be much better soloutions after migration to typescript.... getGlobalStorageProvider().getDataAsBoolean(key, true, value => { this.settingsCache.set(key, value); @@ -45,4 +45,4 @@ export function getGlobalConfiguration() { } return __globalConfig; -} \ No newline at end of file +} From 1914b691d65b1118c7a395cb5809f3c65238791c Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 15:42:06 +0100 Subject: [PATCH 21/29] #48 Fixed watch script --- gulpfile.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gulpfile.js b/gulpfile.js index 0868042..3d74071 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -208,7 +208,7 @@ Watchers gulp.task('watch', (done) => { gulp.watch(`${src.styles}/**/*.scss`, gulp.series('clean:build', 'styles', 'dist:copy', 'dist:zip')) - gulp.watch(`${src.scripts}/**/*.js`, gulp.series('clean:build', 'scripts', 'dist:copy', 'dist:zip')) + gulp.watch(`${src.scripts}/**/*.ts`, gulp.series('clean:build', 'scripts', 'dist:copy', 'dist:zip')) gulp.watch(`${src.images}/**/*`, gulp.series('clean:build', 'images', 'dist:copy', 'dist:zip')) From 5295c3d3ae394fbf543a55bcca18898564146a75 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 15:53:24 +0100 Subject: [PATCH 22/29] #48 Changed time conversion to typescript file --- .../enhancements/{timeConversion.js => timeConversion.ts} | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename src/javascript/enhancements/{timeConversion.js => timeConversion.ts} (100%) diff --git a/src/javascript/enhancements/timeConversion.js b/src/javascript/enhancements/timeConversion.ts similarity index 100% rename from src/javascript/enhancements/timeConversion.js rename to src/javascript/enhancements/timeConversion.ts From a6b7c3ba6fd052296b1e5a6ec8a2bf5032e04ff7 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 16:20:15 +0100 Subject: [PATCH 23/29] #48 Exchanged spacetime with a custom version MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This change only is temporary because of missinng type definitions for spacetime. For more details please see https://github.com/spencermountain/spacetime/issues/255. A pull request to spacetime has been opened. Once this is merged and the fix is published we can switch back. I´ll open an issue for this later. --- package-lock.json | 5 ++--- package.json | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/package-lock.json b/package-lock.json index 264999e..1520892 100644 --- a/package-lock.json +++ b/package-lock.json @@ -11847,9 +11847,8 @@ "dev": true }, "spacetime": { - "version": "6.12.2", - "resolved": "https://registry.npmjs.org/spacetime/-/spacetime-6.12.2.tgz", - "integrity": "sha512-w0St4Q9X8KtuZ/JY8+FM8a4hMrAoNNUWQCt9UQQAUzwk8eDW5wrGh4SaNvEg+9cjLF++vixm6SgJyC6F7ALF/A==" + "version": "github:Serraniel/spacetime#84ae159680732067195b84744e172f4e2497b4ec", + "from": "github:Serraniel/spacetime#bugfix/#255-typescript-constructor-options-missing" }, "sparkles": { "version": "1.0.1", diff --git a/package.json b/package.json index 13d85aa..f38d96f 100644 --- a/package.json +++ b/package.json @@ -34,7 +34,7 @@ "homepage": "https://github.com/Serraniel/AniwatchPlus#readme", "dependencies": { "color": "^3.1.3", - "spacetime": "^6.12.2", + "spacetime": "github:Serraniel/spacetime#bugfix/#255-typescript-constructor-options-missing", "uuid": "^8.3.2" }, "devDependencies": { From b08e38e9154a1e6200072a37f87d06abe8d46bce Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 16:33:02 +0100 Subject: [PATCH 24/29] #48 Added types for timeconversion enhancements. --- src/javascript/enhancements/timeConversion.ts | 30 +++++++++++-------- src/javascript/utils/helpers.ts | 2 +- tsconfig.json | 4 ++- 3 files changed, 21 insertions(+), 15 deletions(-) diff --git a/src/javascript/enhancements/timeConversion.ts b/src/javascript/enhancements/timeConversion.ts index a479759..ce118b6 100644 --- a/src/javascript/enhancements/timeConversion.ts +++ b/src/javascript/enhancements/timeConversion.ts @@ -6,7 +6,7 @@ import * as helper from '../utils/helpers'; const __alteredNodes = []; const DAYS = ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday']; -export function init() { +export function init(): void { getGlobalConfiguration().getProperty(SETTINGS_websiteAutoTimeConversion, value => { if (value) { // The regexp pattern matches anything except the airing page. @@ -20,18 +20,18 @@ export function init() { updateTimestamps(document.documentElement); }, "^/(?!airing).*$"); - core.registerScript(node => { + core.registerScript((node: Node) => { updateTimestamps(node); }, "^/(?!airing).*$"); } }); } -function getSpaceDateTimeFormat(use24Format) { +function getSpaceDateTimeFormat(use24Format: boolean): string { return `${getSpaceDateFormat()} ${getSpaceTimeFormat(use24Format)}`; } -function getSpaceTimeFormat(use24Format) { +function getSpaceTimeFormat(use24Format: boolean): string { if (use24Format) { return '{time-24}'; } @@ -39,11 +39,11 @@ function getSpaceTimeFormat(use24Format) { return '{time}'; } -function getSpaceDateFormat() { +function getSpaceDateFormat(): string { return '{date}. {month-short} {year}'; } -function tryUpdateDateTime(node) { +function tryUpdateDateTime(node: Node): boolean { const REG_DATETIME = /(\d{2}(\/|\.)){2}\d{4} *\d?\d:\d{2}( (AM|PM))?/g; const REG_TIME = /\d?\d:\d{2}/; const REG_AMPM = /\s(am|pm)/i; @@ -81,7 +81,7 @@ function tryUpdateDateTime(node) { let datetime = spacetime(processedStr, 'UTC+1', { dmy: true }); datetime = datetime.goto(spacetime().tz); - let replaceText = datetime.format(getSpaceDateTimeFormat(use24Format)); + let replaceText = String(datetime.format(getSpaceDateTimeFormat(use24Format))); node.textContent = node.textContent.replace(hit, replaceText); }); @@ -89,7 +89,7 @@ function tryUpdateDateTime(node) { return true; } -function tryUpdateDate(node) { +function tryUpdateDate(node: Node): boolean { const REG_DATE = /(\d{2}(\/|\.)){2}\d{4}/g; let hits = Array.from(node.textContent.matchAll(REG_DATE), match => match[0]); @@ -101,7 +101,7 @@ function tryUpdateDate(node) { hits.forEach(hit => { let datetime = spacetime(hit, 'UTC+1', { dmy: true }); datetime = datetime.goto(spacetime().tz); - let replaceText = datetime.format(getSpaceDateFormat()); + let replaceText = String(datetime.format(getSpaceDateFormat())); node.textContent = node.textContent.replace(hit, replaceText); }); @@ -109,7 +109,7 @@ function tryUpdateDate(node) { return true; } -function tryUpdateTime(node) { +function tryUpdateTime(node: Node): boolean { const REG_TIME = /\d?\d:\d{2}( (AM|PM))?/g; const REG_AMPM = /\s(am|pm)/i; @@ -148,7 +148,7 @@ function tryUpdateTime(node) { datetime = datetime.goto('UTC+1'); datetime = datetime.time(processedStr); datetime = datetime.goto(spacetime().tz); - let replaceText = datetime.format(getSpaceTimeFormat(use24Format)); + let replaceText = String(datetime.format(getSpaceTimeFormat(use24Format))); node.textContent = node.textContent.replace(hit, replaceText); @@ -168,7 +168,7 @@ function tryUpdateTime(node) { // if day changed if (dOffset != 0) { - let dayNode = node.parentNode.previousElementSibling; + let dayNode = (node.parentNode as Element)?.previousElementSibling; if (helper.assigned(dayNode)) { for (let i = 0; i < DAYS.length; i++) { if (dayNode.textContent.indexOf(DAYS[i]) >= 0) { @@ -183,13 +183,17 @@ function tryUpdateTime(node) { return true; } -function tryUpdateTimeZone(node) { +function tryUpdateTimeZone(node: Node): boolean { const HINT_UTC = 'UTC+1'; if (node.textContent === HINT_UTC) { let tzMeta = spacetime().timezone(); node.textContent = `${tzMeta.name} (UTC${tzMeta.current.offset >= 0 ? '+' : ''}${tzMeta.current.offset})`; + + return true; } + + return false; } function updateTimestamps(node) { diff --git a/src/javascript/utils/helpers.ts b/src/javascript/utils/helpers.ts index 82a65c5..e9c01fd 100644 --- a/src/javascript/utils/helpers.ts +++ b/src/javascript/utils/helpers.ts @@ -38,7 +38,7 @@ function handleKeyToggle(event: KeyboardEvent, isPressed: boolean) { } } -export function findTextNodes(baseNode) { +export function findTextNodes(baseNode: Node): Array { if (!assigned(baseNode)) { baseNode = document.documentElement; } diff --git a/tsconfig.json b/tsconfig.json index 05d6614..b989c89 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -1,6 +1,8 @@ { "compilerOptions": { // You have to explicitly set @types to get DefinitelyTyped type definitions - "typeRoots": ["node_modules/@types", "node_modules/web-ext-types"] + "typeRoots": ["node_modules/@types", "node_modules/web-ext-types"], + // Default would be DOM,ES6,DOM.Iterable,ScriptHost (see https://www.typescriptlang.org/docs/handbook/compiler-options.html). However the ES 2020 is required for string.prototype.matchAll (see https://stackoverflow.com/a/57298833). + "lib": ["ES2020", "DOM", "DOM.Iterable", "ScriptHost"] } } \ No newline at end of file From 421c191e1da6163d783bd05ab939b23456b241de Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 16:49:18 +0100 Subject: [PATCH 25/29] #48 Better browser detection --- src/javascript/browserApi/storageProvider.ts | 36 ++++++++++++++++---- 1 file changed, 30 insertions(+), 6 deletions(-) diff --git a/src/javascript/browserApi/storageProvider.ts b/src/javascript/browserApi/storageProvider.ts index 31a76bf..61b8ef4 100644 --- a/src/javascript/browserApi/storageProvider.ts +++ b/src/javascript/browserApi/storageProvider.ts @@ -1,5 +1,11 @@ const { assigned } = require("../utils/helpers") +enum BrowserApi { + Unknown, + Chromium, + Firefox, +} + export type ConfigurationStorageBooleanCallback = (value: boolean) => void; export interface ICustomBrowserStorageProvider { @@ -70,18 +76,36 @@ class StorageProviderFirefox implements ICustomBrowserStorageProvider { let __storageProvieder: ICustomBrowserStorageProvider; +function getBrowserApi(): BrowserApi { + if (assigned(chrome)) { + if (assigned(browser)) { + return BrowserApi.Firefox; + } + + return BrowserApi.Chromium; + } + else if (assigned(browser)) { + return BrowserApi.Firefox; + } + + return BrowserApi.Unknown; +} + function createStorageProvider() { - // chrome based browser - // TODO: chrome.app? - // if (assigned(chrome?.app)) { - if (true) { + + let api = getBrowserApi(); + + // chromium + if (api === BrowserApi.Chromium) { __storageProvieder = new StorageProviderChromium(); } // firefox - else { + else if (api === BrowserApi.Firefox) { __storageProvieder = new StorageProviderFirefox(); } - + else { + throw "Unknown browser API. Cannot create storage provider."; + } } export function getGlobalStorageProvider(): ICustomBrowserStorageProvider { From 71c5b6ae55448d1db177a00b1650445502ba2a26 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 16:59:28 +0100 Subject: [PATCH 26/29] #48 Fixes in storageProvider --- src/javascript/browserApi/storageProvider.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/javascript/browserApi/storageProvider.ts b/src/javascript/browserApi/storageProvider.ts index 61b8ef4..0a2f812 100644 --- a/src/javascript/browserApi/storageProvider.ts +++ b/src/javascript/browserApi/storageProvider.ts @@ -1,4 +1,4 @@ -const { assigned } = require("../utils/helpers") +import { assigned } from '../utils/helpers'; enum BrowserApi { Unknown, @@ -77,14 +77,14 @@ class StorageProviderFirefox implements ICustomBrowserStorageProvider { let __storageProvieder: ICustomBrowserStorageProvider; function getBrowserApi(): BrowserApi { - if (assigned(chrome)) { - if (assigned(browser)) { + if (typeof chrome !== 'undefined') { + if (typeof browser !== 'undefined') { return BrowserApi.Firefox; } return BrowserApi.Chromium; } - else if (assigned(browser)) { + else if (typeof browser !== 'undefined') { return BrowserApi.Firefox; } From 96000452cf6ddc6bdd9081fa85b696674c4dd06c Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 16:59:53 +0100 Subject: [PATCH 27/29] #48 Fixed tsconfig for module loading --- tsconfig.json | 1 + 1 file changed, 1 insertion(+) diff --git a/tsconfig.json b/tsconfig.json index b989c89..c7e9ab4 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -1,5 +1,6 @@ { "compilerOptions": { + "esModuleInterop": true, // You have to explicitly set @types to get DefinitelyTyped type definitions "typeRoots": ["node_modules/@types", "node_modules/web-ext-types"], // Default would be DOM,ES6,DOM.Iterable,ScriptHost (see https://www.typescriptlang.org/docs/handbook/compiler-options.html). However the ES 2020 is required for string.prototype.matchAll (see https://stackoverflow.com/a/57298833). From ac26b2b3308ecd9acda56ac99ddb48cc5e4dedf1 Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 17:07:33 +0100 Subject: [PATCH 28/29] #48 Several optimizations for font color enhancements --- src/javascript/enhancements/fontColor.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/javascript/enhancements/fontColor.ts b/src/javascript/enhancements/fontColor.ts index f01435b..9938169 100644 --- a/src/javascript/enhancements/fontColor.ts +++ b/src/javascript/enhancements/fontColor.ts @@ -54,8 +54,7 @@ function tryRegisterObserverForBadge(badge: Node): void { mutations.forEach(mutation => { // prevent recursive calls when our class is added / removed if (mutation.target instanceof Element) { - // TODO: Validate if that still works - if ((mutation.oldValue?.indexOf(DARKCOLOR_CLASS) ?? -1 ^ (mutation.target?.classList?.contains(DARKCOLOR_CLASS) ?? false ? 0 : -1)) === 0) { + if ((mutation.oldValue?.indexOf(DARKCOLOR_CLASS) >= 0 ?? false) !== (mutation.target?.classList?.contains(DARKCOLOR_CLASS) ?? false)) { return; } else { From 3539af5de315fbdf6f99c454f0d745b1203c5f4c Mon Sep 17 00:00:00 2001 From: Serraniel Date: Tue, 29 Dec 2020 17:08:54 +0100 Subject: [PATCH 29/29] =?UTF-8?q?#48=20Removed=20comment=20I=20cannot=20re?= =?UTF-8?q?member=20what=20I=20ment=20with=20=F0=9F=98=82=F0=9F=98=8E?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/javascript/configuration/configuration.ts | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/javascript/configuration/configuration.ts b/src/javascript/configuration/configuration.ts index 2e78d91..64207c6 100644 --- a/src/javascript/configuration/configuration.ts +++ b/src/javascript/configuration/configuration.ts @@ -28,8 +28,6 @@ class Configuration { callback(this.settingsCache.get(key)); } else { - // TODO see comment - // OOOPS // currently all settings are default true. This isn´t a problem but there should be much better soloutions after migration to typescript.... getGlobalStorageProvider().getDataAsBoolean(key, true, value => { this.settingsCache.set(key, value); callback(value);