From 69b12f307566a006ce7e63a2fd1156f60d300b48 Mon Sep 17 00:00:00 2001
From: Mathias Jakobsen <mathias.jakobsen@overleaf.com>
Date: Mon, 12 Jan 2026 12:52:59 +0000
Subject: [PATCH] Merge pull request #30632 from
 overleaf/mj-pdf-dark-mode-tear-down

[web] Tear down pdf-dark-mode feature flag

GitOrigin-RevId: 5f3fa98af8eba70139ec614a3c2e08e380b56de9
---
 services/web/app/src/Features/Project/ProjectController.mjs | 1 -
 .../ide-redesign/contexts/settings-modal-context.tsx        | 5 ++---
 .../pdf-preview/components/pdf-hybrid-theme-button.tsx      | 6 ------
 .../js/features/pdf-preview/components/pdf-preview-pane.tsx | 3 ---
 4 files changed, 2 insertions(+), 13 deletions(-)

diff --git a/services/web/app/src/Features/Project/ProjectController.mjs b/services/web/app/src/Features/Project/ProjectController.mjs
index 00d6a0a3c7..2c1eb2c255 100644
--- a/services/web/app/src/Features/Project/ProjectController.mjs
+++ b/services/web/app/src/Features/Project/ProjectController.mjs
@@ -457,7 +457,6 @@ const _ProjectController = {
       'wf-citations-checker',
       'wf-citations-checker-on-selection',
       'writefull-asymetric-queue-size-per-model',
-      'pdf-dark-mode',
       'email-notifications',
       'editor-redesign-no-opt-out',
     ].filter(Boolean)
diff --git a/services/web/frontend/js/features/ide-redesign/contexts/settings-modal-context.tsx b/services/web/frontend/js/features/ide-redesign/contexts/settings-modal-context.tsx
index c20e39401d..56f0d4e30f 100644
--- a/services/web/frontend/js/features/ide-redesign/contexts/settings-modal-context.tsx
+++ b/services/web/frontend/js/features/ide-redesign/contexts/settings-modal-context.tsx
@@ -87,7 +87,6 @@ export const SettingsModalProvider: FC<React.PropsWithChildren> = ({
   // TODO ide-redesign-cleanup: Rename this field and move it directly into this context
   const { leftMenuShown, setLeftMenuShown } = useLayoutContext()
 
-  const hasDarkModePdf = useFeatureFlag('pdf-dark-mode')
   const hasEmailNotifications = useFeatureFlag('email-notifications')
   const noNewEditorOptOut = useFeatureFlag('editor-redesign-no-opt-out')
 
@@ -213,7 +212,7 @@ export const SettingsModalProvider: FC<React.PropsWithChildren> = ({
               {
                 key: 'pdfDarkMode',
                 component: <DarkModePdfSetting />,
-                hidden: overallTheme === 'light-' || !hasDarkModePdf,
+                hidden: overallTheme === 'light-',
               },
               {
                 key: 'fontSize',
@@ -268,7 +267,7 @@ export const SettingsModalProvider: FC<React.PropsWithChildren> = ({
         href: '/user/subscription',
       },
     ],
-    [t, overallTheme, hasDarkModePdf, hasEmailNotifications, noNewEditorOptOut]
+    [t, overallTheme, hasEmailNotifications, noNewEditorOptOut]
   )
 
   const settingsTabs = useMemo(
diff --git a/services/web/frontend/js/features/pdf-preview/components/pdf-hybrid-theme-button.tsx b/services/web/frontend/js/features/pdf-preview/components/pdf-hybrid-theme-button.tsx
index 2c2e167c12..8e5af5ea81 100644
--- a/services/web/frontend/js/features/pdf-preview/components/pdf-hybrid-theme-button.tsx
+++ b/services/web/frontend/js/features/pdf-preview/components/pdf-hybrid-theme-button.tsx
@@ -2,14 +2,12 @@ import OLTooltip from '@/shared/components/ol/ol-tooltip'
 import { useCallback, useId } from 'react'
 import { useTranslation } from 'react-i18next'
 import { useDetachCompileContext as useCompileContext } from '../../../shared/context/detach-compile-context'
-import { useFeatureFlag } from '@/shared/context/split-test-context'
 import OLIconButton from '@/shared/components/ol/ol-icon-button'
 import { useIsNewEditorEnabled } from '@/features/ide-redesign/utils/new-editor-utils'
 
 export const PdfHybridThemeButton = () => {
   const id = useId()
   const { t } = useTranslation()
-  const splitTestEnabled = useFeatureFlag('pdf-dark-mode')
   const usesNewEditor = useIsNewEditorEnabled()
   const {
     pdfViewer,
@@ -28,10 +26,6 @@ export const PdfHybridThemeButton = () => {
     return null
   }
 
-  if (!splitTestEnabled) {
-    return null
-  }
-
   if (activeOverallTheme !== 'dark') {
     return null
   }
diff --git a/services/web/frontend/js/features/pdf-preview/components/pdf-preview-pane.tsx b/services/web/frontend/js/features/pdf-preview/components/pdf-preview-pane.tsx
index 604c9a5b61..e27814e668 100644
--- a/services/web/frontend/js/features/pdf-preview/components/pdf-preview-pane.tsx
+++ b/services/web/frontend/js/features/pdf-preview/components/pdf-preview-pane.tsx
@@ -14,7 +14,6 @@ import importOverleafModules from '../../../../macros/import-overleaf-module.mac
 import PdfCodeCheckFailedBanner from '@/features/ide-redesign/components/pdf-preview/pdf-code-check-failed-banner'
 import getMeta from '@/utils/meta'
 import NewPdfLogsViewer from '@/features/ide-redesign/components/pdf-preview/pdf-logs-viewer'
-import { useFeatureFlag } from '@/shared/context/split-test-context'
 
 function PdfPreviewPane() {
   const {
@@ -25,9 +24,7 @@ function PdfPreviewPane() {
   } = useCompileContext()
   const { compileTimeout } = getMeta('ol-compileSettings')
   const usesNewEditor = useIsNewEditorEnabled()
-  const inDarkModePdfSplitTest = useFeatureFlag('pdf-dark-mode')
   const darkModePdf =
-    inDarkModePdfSplitTest &&
     usesNewEditor &&
     pdfViewer === 'pdfjs' &&
     activeOverallTheme === 'dark' &&