Page MenuHomePhabricator

[native] Introduce designSystemColors to colors.js
ClosedPublic

Authored by ashoat on May 15 2023, 10:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:04 PM
Unknown Object (File)
Thu, Apr 11, 12:45 PM
Unknown Object (File)
Thu, Apr 11, 12:45 PM
Unknown Object (File)
Mar 27 2024, 11:53 PM
Unknown Object (File)
Mar 7 2024, 7:17 AM
Unknown Object (File)
Feb 21 2024, 3:57 AM
Unknown Object (File)
Feb 21 2024, 2:41 AM
Unknown Object (File)
Feb 21 2024, 1:35 AM
Subscribers

Details

Summary

Following discussion offline and in D7740, we want to make native/themes/colors.js look more like web/theme.css, where the design system colors are defined at the top of the page in their own struct, and the domain-specific usages all references the design system colors.

There are many cases of colors that are using in the mobile app that are NOT in the design system. We'll sync with @ted on this and come back to it later.

Test Plan

Flow, careful reading

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Sweet, I think this will make colors.js much easier to work w/ in the future.

This revision is now accepted and ready to land.May 15 2023, 11:13 AM
This revision is now accepted and ready to land.May 15 2023, 11:14 AM