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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the color alpha syntax to be 50% #182

Merged
merged 1 commit into from Dec 13, 2022

Conversation

kizu
Copy link
Contributor

@kizu kizu commented Nov 16, 2022

Noticed an issue with the Example 15 and https://design-tokens.github.io/community-group/format/#example-16: Per the spec text, the 24 bit color value is expected to be transformed into a color with the alpha value of 50%, while 88 would be actually around 53%, being a common misconception when writing the component in 8bit format.

This PR changes any mentioned #00000088 colors to #00000080 to match the intent and fix the examples.

@netlify
Copy link

netlify bot commented Nov 16, 2022

Deploy Preview for dtcg-tr ready!

Name Link
🔨 Latest commit 0e815eb
🔍 Latest deploy log https://app.netlify.com/sites/dtcg-tr/deploys/6374f06f9f2b210008d7d649
😎 Deploy Preview https://deploy-preview-182--dtcg-tr.netlify.app/format
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Member

@c1rrus c1rrus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good spot! Thank you so much for contributing these corrections!

@kaelig kaelig merged commit 6c0a7c6 into design-tokens:main Dec 13, 2022
github-actions bot added a commit that referenced this pull request Dec 13, 2022
SHA: 6c0a7c6
Reason: push, by kaelig

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Dec 13, 2022
SHA: 6c0a7c6
Reason: push, by kaelig

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Dec 13, 2022
SHA: 6c0a7c6
Reason: push, by kaelig

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants