Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 Move all colors to use Display-P3 color-space #16

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

mdb1
Copy link
Owner

@mdb1 mdb1 commented Sep 1, 2021

馃憢 Intro

The method to retrieve the hexadecimal of the colors was failing to retrieve the correct hexa for Apple's semantic colors, so this PR moves all the colors to use Display P3 as their color-space

馃 What changed?

  • Fixed bugs with hexadecimals
  • Fixed the tests
  • Added consistency by always using the same color-space

* Fix bugs with hexadecimals
* Fix the tests
* Add consistency by always using the same color-space
@mdb1 mdb1 added the bug Something isn't working label Sep 1, 2021
@mdb1 mdb1 self-assigned this Sep 1, 2021
@mdb-danger-bot
Copy link

Messages
馃摉 馃帀 The PR added 173 and removed 144 lines. 馃梻 18 files changed.

Generated by 馃毇 Danger Swift against 176774b

@mdb1 mdb1 merged commit e7c16d4 into main Sep 1, 2021
@mdb1 mdb1 deleted the bug/fix-colors-hexas branch September 1, 2021 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants