Skip to content

Conversation

@canova
Copy link
Member

@canova canova commented Jan 27, 2026

While talking with Markus, we decided to hide this dark mode checkbox for now, and move the toggle to the devtools console. This will allow us to deploy the frontend sooner than later.
You can use window.toggleDarkMode() to toggle the dark mode.

The follow-up would be to:

  • Use the system default for the dark/light mode.
  • Have a toggle to switch between system/light/dark.

Main branch / deploy preview

@canova canova changed the title Darkmode checkbox Move the dark mode toggle to devtools console Jan 27, 2026
@canova canova requested review from fatadel and mstange January 27, 2026 13:11
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.63%. Comparing base (800ffe0) to head (450121b).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/utils/window-console.ts 14.28% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5783      +/-   ##
==========================================
- Coverage   85.67%   85.63%   -0.05%     
==========================================
  Files         319      319              
  Lines       31197    31197              
  Branches     8585     8497      -88     
==========================================
- Hits        26728    26714      -14     
- Misses       4039     4053      +14     
  Partials      430      430              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@mstange mstange left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@arai-a We're doing this because I forgot to ask for the checkbox label to be localizeable and we'd just like to deploy what we have now. Nazim is planning on adding a prettier toggle button as a follow-up.

We discussed that we would like to have a different switch, but to be
able to deploy the current patches so far, removing the current UI.

After the previous commit, we can now toggle the dark mode from the
devtools console.
It looks like this is the remnants of the previous implementation that
was reverted.
@arai-a
Copy link
Contributor

arai-a commented Jan 27, 2026

Thank you for taking care of the feature :)
Let me know if there's anything I can do.

@canova canova force-pushed the darkmode-checkbox branch from f481a2f to 450121b Compare January 27, 2026 16:06
@canova canova enabled auto-merge January 27, 2026 16:07
@canova canova merged commit fb81edd into firefox-devtools:main Jan 27, 2026
19 checks passed
@canova canova mentioned this pull request Jan 27, 2026
canova added a commit that referenced this pull request Jan 27, 2026
Lots of exciting changes 🎉:

[arai-a] Put radio buttons into labels (#5738)
[DaniPopes] Update comment for "unique-string" (#5741)
[Karan Pradhan] Hide tooltip filter button in non-sticky tooltips and
add hideFilterButton tests (#5718)
[arai-a] Add a menu to copy the Marker Table as text (#5732)
[arai-a] Make the entire list item clickable for the "Full Range"
(#5742)
[Markus Stange] Move symbol table demangling out of SymbolStore into
SymbolProvider (#5746)
[Markus Stange] Remove SVG asset imports from profile-data.ts (#5747)
[arai-a] Do not apply sticky tooltip on double click (#5754)
[arai-a] Skip the ChartCanvas redraw on the Viewport's internal default
state usage (#5744)
[Markus Stange] Stop blindly extracting uint8array.buffer after calling
compress() (#5753)
[Markus Stange] In the assembly view state, refer to the current symbol
by index (#5755)
[Markus Stange] Fix "scroll to hotspot" functionality in the source view
+ assembly view (#5759)
[Markus Stange] Keep the colorField markerSchema field when processing
profiles in the gecko format (#5760)
[Markus Stange] Implement dark mode (#5740)
[Markus Stange] Fix light-mode colors (#5765)
[Markus Stange] Tweak dark mode colours. (#5767)
[Nazım Can Altınova] Enable some basic type-aware lints (#5775)
[Markus Stange] Allow seeing different assembly code for the same
function (#5349)
[fatadel] Refine tree view a11y (#5779)
[fatadel] Align double-click behavior of stack chart with flame graph
(#5782)
[Markus Stange] Split gz.ts properly into node and browser variants
(#5764)
[Markus Stange] Simplify and optimize the computation of per-call-node
line and address timings (#5770)
[Nazım Can Altınova] Move the dark mode toggle to devtools console
(#5783)
[Nazım Can Altınova] 🔃 Sync: l10n -> main (Jan 27, 2026) (#5785)
[Nazım Can Altınova] Improve Chrome importer marker payload logic
(#5717)
[Markus Stange] Add a Focus Self transform (#5774)
[Nazım Can Altınova] Enable the Turkish locale in production (#5786)


And huge thanks to our localizers 🎉 :

be: Mikalai Udodau
de: Ger
de: Michael Köhler
el: Jim Spentzos
en-CA: chutten
en-CA: Saurabh
en-GB: Ian Neal
en-GB: Saurabh
es-CL: ravmn
fy-NL, nl: Fjoerfoks
fr: Skywarp
fr: Théo Chevalier
fur: Fabio Tomat
fy-NL: Fjoerfoks
ia: Melo46
it: Francesco Lodolo [:flod]
nl: Fjoerfoks
nl: Mark Heijl
pt-BR: Marcelo Ghelman
ru: berry
ru: Valery Ledovskoy
sv-SE: Andreas Pettersson
tr: Grk
zh-CN: Olvcpr423
zh-CN: wxie
zh-TW: Pin-guang Chen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants