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
F2945807: D25396.1738362295.diff
Thu, Jan 30, 22:24
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

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
Branch
css-source-code-view
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 765
Build 765: arc lint + arc unit

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