From e4c825e58305481c4a13c17d18d33fb1267d9ea1 Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Wed, 20 Jul 2022 19:21:32 +0200 Subject: files: Fix colormode detection for filelist When system default color theme is selected for theming, the `enabledThemes` array is empty or just contains on entry `'default'`, in this case the color theme has to be retrieved from the browser to ensure text like the modified date is readable. This fixes #33298 Signed-off-by: Ferdinand Thiessen --- apps/files/js/filelist.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'apps/files') diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js index 42afe792142..d745f3368a5 100644 --- a/apps/files/js/filelist.js +++ b/apps/files/js/filelist.js @@ -1787,8 +1787,11 @@ td.append(linkElem); tr.append(td); - var enabledThemes = window.OCA?.Theming?.enabledThemes || [] - var isDarkTheme = enabledThemes.join('').indexOf('dark') !== -1 + const enabledThemes = window.OCA?.Theming?.enabledThemes || [] + // Check enabled themes, if system default is selected check the browser + const isDarkTheme = (enabledThemes.length === 0 || enabledThemes[0] === 'default') + ? window.matchMedia('(prefers-color-scheme: dark)').matches + : enabledThemes.join('').indexOf('dark') !== -1 try { var maxContrastHex = window.getComputedStyle(document.documentElement) -- cgit v1.2.3