Page MenuHomePhorge

Fix source code viewer background color when using Dark Mode in Diffusion
ClosedPublic

Authored by bob on Aug 16 2023, 14:41.
Tags
None
Referenced Files
F2945430: D25396.1738317171.diff
Thu, Jan 30, 09:52
F2942964: D25396.1738150676.diff
Tue, Jan 28, 11:37
F2940506: D25396.1738078995.diff
Mon, Jan 27, 15:43
F2938307: D25396.1737994128.diff
Sun, Jan 26, 16:08
F2935279: D25396.1737840757.diff
Fri, Jan 24, 21:32
F2934783: D25396.1737803315.diff
Fri, Jan 24, 11:08
F2928519: D25396.1737650448.diff
Wed, Jan 22, 16:40
F2902177: D25396.1737297067.diff
Sat, Jan 18, 14:31

Details

Summary

This CSS fix replaces the hard coded white background value by the diff.background CSS variable. This is a proposal
to address this issue but it may be a good idea to create a different CSS variable (for instance source.background)
in order to avoid any potential side effect in the future.

BeforeAfter
screenshot-phorge-bug-darkmode-source-view-before.png (536×1 px, 30 KB)
screenshot-phorge-bug-darkmode-source-view-after.png (536×1 px, 49 KB)

Fix : T15056

Test Plan
  • Flush all Phorge caches
  • Sign in
  • Open a diffusion repository
  • Open any file
  • Check that viewer background color is consistent with the theme.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bob requested review of this revision.Aug 16 2023, 14:41

Updating D25396: Include updated celerity map file

Nice thaaanks

I agree that a follow-up change could improve even more the situation, but this just looks great both in white mode, and in dark mode.

Tested, no nuclear implosions, and celerity is updated, I also brushed my teeth correctly.

sgtm

This revision is now accepted and ready to land.Aug 17 2023, 08:25