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(tokens): DLT-2053 android tokens color value #504

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

ninamarina
Copy link
Contributor

@ninamarina ninamarina commented Sep 20, 2024

Fix Android color tokens

Obligatory GIF (super important!)

Obligatory GIF

🛠️ Type Of Change

These types will increment the version number on release:

  • Fix
  • Feature
  • Performance Improvement
  • Refactor

📖 Jira Ticket

DLT-2053

📖 Description

Fixes the color tokens output for Android. As an example you can check out dtColorForegroundMuted, that before had the value Color(0xff000000) and now has the correct Color(0x80222222).

💡 Context

It started working wrong when removing token-transformer.

📷 Screenshots / GIFs

Before

image

Now

image

@ninamarina ninamarina changed the title fix(tokens): android tokens color value fix(tokens): DLT-2053 android tokens color value Sep 20, 2024
Copy link

Please add either the visual-test-ready or no-visual-test label to this PR depending on whether you want to run visual tests or not.
It is recommended to run visual tests if your PR changes any UI. ‼️

Copy link

✔️ Deploy previews ready!
😎 Dialtone preview: https://dialtone.dialpad.com/deploy-previews/pr-504/
😎 Dialtone-vue 2 preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-504/
😎 Dialtone-vue 3 the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-504/

@ninamarina ninamarina marked this pull request as ready for review September 20, 2024 18:36
Copy link
Contributor

@braddialpad braddialpad left a comment

Choose a reason for hiding this comment

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

Looks good functionally. Can probably simplify the code a little.

@ninamarina ninamarina merged commit 368c9db into staging Sep 23, 2024
10 checks passed
@ninamarina ninamarina deleted the fix/android-tokens branch September 23, 2024 13:47
juliodialpad pushed a commit that referenced this pull request Sep 23, 2024
# [9.77.0-beta.1](dialtone/v9.76.0...dialtone/v9.77.0-beta.1) (2024-09-23)

### Bug Fixes

* NO-JIRA merge staging into beta ([#478](#478)) ([09f1165](09f1165))
* **Tokens:** DLT-2053 android tokens color value ([#504](#504)) ([368c9db](368c9db))

### Code Refactoring

* **Badge, Emoji Tabset, Empty State:** remove DtIcon ([#481](#481)) ([0fa1280](0fa1280))

### Features

* **Avatar:** DLT-1916 remove dt-icon ([#466](#466)) ([6ef90db](6ef90db))
* **Avatar:** DLT-1916 remove dt-icon from Avatar vue 2 ([#474](#474)) ([e8600c3](e8600c3))
* **Feed Item Pill:** DLT-1916 remove dt-icon from Feed Item Pill ([#489](#489)) ([bde73c8](bde73c8))
* **Keyboard Shortcut To Unread Pill:** DLT-1916 remove dt-icon ([#482](#482)) ([277ae13](277ae13))
* **Message Input:** DLT-1916 remove dt-icon from message input ([#490](#490)) ([3215116](3215116))
juliodialpad pushed a commit that referenced this pull request Sep 23, 2024
## [1.36.3-beta.1](dialtone-tokens/v1.36.2...dialtone-tokens/v1.36.3-beta.1) (2024-09-23)

### Bug Fixes

* NO-JIRA merge staging into beta ([#478](#478)) ([09f1165](09f1165))
* **Tokens:** DLT-2053 android tokens color value ([#504](#504)) ([368c9db](368c9db))
juliodialpad pushed a commit that referenced this pull request Sep 24, 2024
## [9.76.1](dialtone/v9.76.0...dialtone/v9.76.1) (2024-09-24)

### Bug Fixes

* **Tokens:** DLT-2053 android tokens color value ([#504](#504)) ([368c9db](368c9db))
juliodialpad pushed a commit that referenced this pull request Sep 24, 2024
## [1.36.3](dialtone-tokens/v1.36.2...dialtone-tokens/v1.36.3) (2024-09-24)

### Bug Fixes

* **Tokens:** DLT-2053 android tokens color value ([#504](#504)) ([368c9db](368c9db))
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.

2 participants