From adf5f7da6416b0e09839057cd10197105eb6da3d Mon Sep 17 00:00:00 2001 From: Kavin <20838718+FireMasterK@users.noreply.github.com> Date: Wed, 16 Nov 2022 18:51:56 +0000 Subject: [PATCH] Don't show alerts in some scenarios. --- src/components/ShareModal.vue | 4 ++-- src/components/VideoPlayer.vue | 4 ++-- src/components/WatchVideo.vue | 2 +- src/main.js | 4 ++-- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/components/ShareModal.vue b/src/components/ShareModal.vue index 17429913..10b6c74d 100644 --- a/src/components/ShareModal.vue +++ b/src/components/ShareModal.vue @@ -66,8 +66,8 @@ export default { } }, onChange() { - this.setPreference("shareWithTimeCode", this.withTimeCode); - this.setPreference("shareAsPipedLink", this.pipedLink); + this.setPreference("shareWithTimeCode", this.withTimeCode, true); + this.setPreference("shareAsPipedLink", this.pipedLink, true); }, }, computed: { diff --git a/src/components/VideoPlayer.vue b/src/components/VideoPlayer.vue index d85c44ca..57b5ede1 100644 --- a/src/components/VideoPlayer.vue +++ b/src/components/VideoPlayer.vue @@ -372,13 +372,13 @@ export default { }); videoEl.addEventListener("volumechange", () => { - this.setPreference("volume", videoEl.volume); + this.setPreference("volume", videoEl.volume, true); }); videoEl.addEventListener("ratechange", e => { const rate = videoEl.playbackRate; if (rate > 0 && !isNaN(videoEl.duration) && !isNaN(videoEl.duration - e.timeStamp / 1000)) - this.setPreference("rate", rate); + this.setPreference("rate", rate, true); }); videoEl.addEventListener("ended", () => { diff --git a/src/components/WatchVideo.vue b/src/components/WatchVideo.vue index adbbb66a..e436d4f0 100644 --- a/src/components/WatchVideo.vue +++ b/src/components/WatchVideo.vue @@ -383,7 +383,7 @@ export default { return this.fetchJson(this.apiUrl() + "/comments/" + this.getVideoId()); }, onChange() { - this.setPreference("autoplay", this.selectedAutoPlay); + this.setPreference("autoplay", this.selectedAutoPlay, true); }, async getVideoData() { await this.fetchVideo() diff --git a/src/main.js b/src/main.js index c4596ac0..fb15e695 100644 --- a/src/main.js +++ b/src/main.js @@ -118,11 +118,11 @@ const mixin = { purifyHTML(original) { return DOMPurify.sanitize(original); }, - setPreference(key, value) { + setPreference(key, value, disableAlert = false) { try { localStorage.setItem(key, value); } catch { - alert(this.$t("info.local_storage")); + if (!disableAlert) alert(this.$t("info.local_storage")); } }, getPreferenceBoolean(key, defaultVal) {