Page MenuHomePhabricator

Add an Experimental Dark Mode to Phabricator
ClosedPublic

Authored by chad on Jul 16 2017, 7:47 PM.
Tags
None
Referenced Files
F14401311: D18227.id.diff
Sun, Dec 22, 9:51 PM
Unknown Object (File)
Sat, Dec 21, 1:49 PM
Unknown Object (File)
Tue, Dec 17, 10:36 AM
Unknown Object (File)
Tue, Dec 17, 6:15 AM
Unknown Object (File)
Fri, Dec 13, 9:03 PM
Unknown Object (File)
Fri, Dec 13, 4:26 PM
Unknown Object (File)
Thu, Dec 12, 7:47 AM
Unknown Object (File)
Thu, Dec 12, 2:29 AM
Subscribers

Details

Summary

Mostly this is an exercise to clean up our CSS and Celerity processor by making sure all important color decisions are generatable. It's somewhat resonable to use if you don't review code. Posting it up here mostly so I don't lose the work.

Test Plan

Visit lots and lots of pages with dark mode on and off.

Diff Detail

Repository
rP Phabricator
Branch
dark-mode (branched from master)
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningwebroot/rsrc/css/phui/phui-form.css:108TXT3Line Too Long
Unit
Tests Passed
Build Status
Buildable 17712
Build 23779: Run Core Tests
Build 23778: arc lint + arc unit

Event Timeline

I think this is pretty reasonable on the technical side given that we've already built this stuff for general maintainability/consistency and accessibility if you want to move forward with it. Two broad thoughts:

  • This makes T12311 slightly harder since there's a bit more stuff to unpack if we're going to offer "Dark Mode, Larger Fonts, Deuteranopia" as options.
  • T5701 might be a pathway forward on the highlighting stuff.

One thing we could do is add, e.g., tabs or buttons something on UIExamples to let you toggle between themes without having to go mangle your settings.

The short TODO on this is:

  • Highlight colors (like light yellow) maybe need to be individual variables.
  • Diffs need work.
  • Setup issues are hard coded.

But overall it came together faster and cleaner than I expected for just goofing around. I'd be OK landing this as it's labeled "experimental". Or I can pull the celerity processor out into a separate diff.

chad retitled this revision from [WIP] Add an Experimental Dark Mode to Phabricator to Add an Experimental Dark Mode to Phabricator.Jul 17 2017, 6:10 PM

This is probably 96% complete. Some random bugs like setting workboard colors or conpherence rooms, but I'm going to put my fingers in my ears about them.

This revision is now accepted and ready to land.Jul 18 2017, 12:08 PM

I think the "Experimental" label is fine, users can't complain to me anymore anyway. :) :) :) 😸 :) :) 11oneone

This revision was automatically updated to reflect the committed changes.