Use FF detection not for P3 but for error message

evilmartians · Jan 17, 2024 · 2a56c80 · 2a56c80
1 parent 8d7e866
commit 2a56c80
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/stores/support.ts b/stores/support.ts
@@ -11,10 +11,7 @@ export let support = map<SupportValue>({
 })
 
 if (typeof window !== 'undefined') {
-  // FF can’t display P3 yet. Remove when this issue will be resolved.
-  // https://bugzilla.mozilla.org/show_bug.cgi?id=1626624
-  let isFF = navigator.userAgent.toLowerCase().includes('firefox')
-  if (CSS.supports('color', 'color(display-p3 1 1 1)') && !isFF) {
+  if (CSS.supports('color', 'color(display-p3 1 1 1)')) {
     let media = window.matchMedia('(color-gamut:p3)')
     support.set({ oklch: true, p3: media.matches })
     media.addEventListener('change', () => {

diff --git a/view/sample/index.ts b/view/sample/index.ts
@@ -10,7 +10,12 @@ let fallbackNote = document.querySelector<HTMLButtonElement>(
   '.sample_fallback .sample_note'
 )!
 
-let postfix = support.get().oklch ? ' on this monitor' : ' in this browser'
+// FF can’t display P3 yet. Remove when this issue will be resolved.
+// https://bugzilla.mozilla.org/show_bug.cgi?id=1626624
+let isFF = navigator.userAgent.toLowerCase().includes('firefox')
+
+let postfix =
+  !isFF && support.get().oklch ? ' on this monitor' : ' in this browser'
 
 visible.subscribe(({ fallback, real, space }) => {
   sample.classList.toggle('is-srgb', space === 'srgb')