Skip to content

Fix dark mode contrast of the warning icons in the publish panel - #6280

Merged
canova merged 2 commits into
firefox-devtools:mainfrom
canova:push-uztkvotquvoo
Aug 24, 2026
Merged

Fix dark mode contrast of the warning icons in the publish panel#6280
canova merged 2 commits into
firefox-devtools:mainfrom
canova:push-uztkvotquvoo

Conversation

@canova

@canovacanova commented Aug 24, 2026

Copy link
Copy Markdown
Member

Main | Deploy preview

Fixes#6279.

The warning indicators next to the publish panel checkboxes were img elements using warning.svg, which has a hardcoded black fill and stayed black on dark backgrounds.

This commit renders them as spans with a background image instead, so they can switch to the light variant (warning-light.svg, already used by the photon message bars) in dark mode.

See the issue for the before image. This is the picture after:
Screenshot 2026-08-24 at 10 35 06

STR:

  • Open a private window
  • Capture a profile
  • Open the sharing popup to see the icon.

The warning indicators next to the publish panel checkboxes were img
elements using warning.svg, which has a hardcoded black fill and stayed
black on dark backgrounds.
This commit renders them as spans with a background image instead, so
they can switch to the light variant (warning-light.svg, already used by
the photon message bars) in dark mode.
@canova
canova requested review from fatadel and mstangeAugust 24, 2026 08:46
@codecov

codecovBot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.79%. Comparing base (9203f11) to head (81694a1).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #6280 +/- ##
=======================================
Coverage 83.79% 83.79% =======================================
Files 350 350 Lines 37583 37584 +1 Branches 10459 10459 =======================================
+ Hits 31492 31493 +1 
Misses 5664 5664 Partials 427 427 

☔ View full report in Codecov by Harness.
📢 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.

@canova
canova enabled auto-merge (squash) August 24, 2026 16:35
@canova
canova merged commit b58a123 into firefox-devtools:mainAug 24, 2026
21 checks passed
@fatadelfatadel mentioned this pull request Sep 3, 2026
fatadel added a commit that referenced this pull request Sep 3, 2026
Changes:
[Nazım Can Altınova] Update oxfmt 0.59.0 → 0.63.0 (major) (#6262)
[Andrew Creskey] Show which network requests were prefetched (#6259)
[Florian Quèze] profiler-cli: document the marker field:value search
syntax (#6265)
[Florian Quèze] profiler-cli: report one time base for text and JSON
output (#6266)
[Florian Quèze] profiler-cli: avoid a stack overflow on large marker
threads (#6264)
[Markus Stange] Give the frameTable a lib column (#6258)
[Florian Quèze] profiler-cli: accept --limit 0 as unlimited, and make
truncation loud (#6267)
[Florian Quèze] profiler-cli: report the network request count the
filters ran against (#6274)
[Nazım Can Altınova] Fix dark mode contrast of the warning icons in the
publish panel (#6280)
[Nazım Can Altınova] Extract the publish panel warning indicator into a
small component (#6282)
[Markus Stange] More typed arrays in the FrameTable (and a flags column)
(#6173)
[Nazım Can Altınova] Improve the profiler-cli publish script and
document the whole deployment in a better way (#6260)
[fatadel] 🔃 Sync: l10n -> main (September 3, 2026) (#6301)
[fatadel] Bump profiler-cli version to 0.9.0 (#6302)
And special thanks to our localizers:
es-CL: ravmn
nl: Mark Heijl
sv-SE: Andreas Pettersson
sv-SE: Luna Jernberg
sv-SE: Peter Kihlstedt
tr: Selim Şumlu
tr: giray
zh-CN: 高乐喆
Sign up for freeto 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.

The PII warning icon in the profile sharing popup doesn't have the right contrats in the dark mode

2 participants

@canova@mstange