Skip to content

Commit

Permalink
ios: Update "Brand" colorset with new brand color!
Browse files Browse the repository at this point in the history
This is the RGBA representation of `hsl(222, 99%, 69%)`, which we've
set as the BRAND_COLOR value in the JavaScript layer. I would put
this in a code comment, but code comments aren't supported in JSON,
which is unfortunate.

We introduced this iOS colorset in 86843a1, to help us get to a
world where it's easier to have the native iOS color definitions
match those in the JavaScript layer.

We haven't yet completed that work by using RN's new PlatformColor
API (which is available to us now because we're using RN v0.63). But
anyway, now that we've started using the new brand color in the
JavaScript layer, we should change this one to match.

Checking zulip#4279 again, we didn't add a counterpart native Android
file that defines the brand color, and I didn't find an existing
one, so I expect we don't need to make this kind of change for
Android.
  • Loading branch information
chrisbobbe committed Apr 6, 2021
1 parent d195454 commit a2e7f62
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions ios/ZulipMobile/Assets.xcassets/Brand.colorset/Contents.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@
"color-space" : "srgb",
"components" : {
"alpha" : "1.000",
"blue" : "0xAF",
"green" : "0xC2",
"red" : "0x51"
"blue" : "0xFE",
"green" : "0x91",
"red" : "0x62"
}
},
"idiom" : "universal"
Expand Down

0 comments on commit a2e7f62

Please sign in to comment.