From 211a45b6887c7923089b0f331e4a6e1657bcd44b Mon Sep 17 00:00:00 2001 From: Henry Jameson Date: Sun, 22 Dec 2024 15:07:20 +0200 Subject: [PATCH 01/13] improve robustness of appearance tab --- src/components/settings_modal/tabs/appearance_tab.js | 11 ++++++++++- .../settings_modal/tabs/theme_tab/theme_tab.js | 2 +- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/src/components/settings_modal/tabs/appearance_tab.js b/src/components/settings_modal/tabs/appearance_tab.js index 7ea4a09ad..927b402d1 100644 --- a/src/components/settings_modal/tabs/appearance_tab.js +++ b/src/components/settings_modal/tabs/appearance_tab.js @@ -119,7 +119,16 @@ const AppearanceTab = { updateIndex('theme').then(themes => { themes.forEach(([key, themePromise]) => themePromise.then(data => { - this.availableThemesV2.push({ key, data, name: data.name, version: 'v2' }) + console.log('DEBUG', data) + if (!data) { + console.warn(`Theme with key ${key} is empty or malformed`) + } else if (Array.isArray(data)) { + console.warn(`Theme with key ${key} is a v1 theme and should be moved to static/palettes/index.json`) + } else if (!data.source && !data.theme) { + console.warn(`Theme with key ${key} is malformed`) + } else { + this.availableThemesV2.push({ key, data, name: data.name, version: 'v2' }) + } })) }) diff --git a/src/components/settings_modal/tabs/theme_tab/theme_tab.js b/src/components/settings_modal/tabs/theme_tab/theme_tab.js index e534a7773..80768fe75 100644 --- a/src/components/settings_modal/tabs/theme_tab/theme_tab.js +++ b/src/components/settings_modal/tabs/theme_tab/theme_tab.js @@ -133,7 +133,7 @@ export default { Object .values(themesIndex) .forEach(themeFunc => { - themeFunc().then(themeData => this.availableStyles.push(themeData)) + themeFunc().then(themeData => themeData && this.availableStyles.push(themeData)) }) }) }, From 69940b7561ce2ddbbfe5393c62a0c79d67c6c64f Mon Sep 17 00:00:00 2001 From: Henry Jameson Date: Sun, 22 Dec 2024 15:08:42 +0200 Subject: [PATCH 02/13] lint EVERYTHING --- src/App.vue | 2 +- .../component_preview/component_preview.vue | 9 +++-- .../contrast_ratio/contrast_ratio.vue | 36 +++++++++++++----- src/components/font_control/font_control.vue | 6 +-- src/components/login_form/login_form.vue | 4 +- .../opacity_input/opacity_input.vue | 2 +- .../settings_modal/tabs/appearance_tab.vue | 16 ++++---- .../settings_modal/tabs/filtering_tab.vue | 2 +- .../settings_modal/tabs/general_tab.vue | 2 +- .../tabs/style_tab/style_tab.vue | 37 ++++++++++++------- .../tabs/style_tab/virtual_directives_tab.vue | 18 ++++----- src/components/timeline/timeline.vue | 5 ++- 12 files changed, 88 insertions(+), 51 deletions(-) diff --git a/src/App.vue b/src/App.vue index 57c32cbd2..762e18515 100644 --- a/src/App.vue +++ b/src/App.vue @@ -70,7 +70,7 @@ - + diff --git a/src/components/component_preview/component_preview.vue b/src/components/component_preview/component_preview.vue index 6795d393a..6aee90a2d 100644 --- a/src/components/component_preview/component_preview.vue +++ b/src/components/component_preview/component_preview.vue @@ -32,8 +32,8 @@ >