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: extra quotes on font tokens #151

Merged
merged 1 commit into from
Feb 13, 2024
Merged

fix: extra quotes on font tokens #151

merged 1 commit into from
Feb 13, 2024

Conversation

braddialpad
Copy link
Contributor

fix: extra quotes on font tokens

Obligatory GIF (super important!)

Obligatory GIF

๐Ÿ› ๏ธ Type Of Change

These types will increment the version number on release:

  • Fix

๐Ÿ“– Jira Ticket

https://dialpad.atlassian.net/browse/DLT-1550

๐Ÿ“– Description

  • Removed the sd-transforms font transform as our font families are already surrounded in quotes
  • Fixed issue with transforms being incorrectly excluded
  • Updated sd-transforms to latest

๐Ÿ’ก Context

Noticed in product that font families with spaces in them from dialtone were not being applied at all due to incorrect syntax '"double quotes"' around them like this.

๐Ÿ“ Checklist

For all PRs:

  • I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
  • I have reviewed my changes.

๐Ÿ”ฎ Next Steps

Release and update in product, remove prior hack that was put in product to fix this.

@braddialpad braddialpad self-assigned this Feb 12, 2024
@braddialpad braddialpad added the no-visual-test Add this tag when the PR does not need visual testing label Feb 12, 2024
Copy link

โœ”๏ธ Deploy previews ready!
๐Ÿ˜Ž Dialtone preview: https://dialtone.dialpad.com/deploy-previews/pr-151/
๐Ÿ˜Ž Dialtone-vue 2 preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-151/
๐Ÿ˜Ž Dialtone-vue 3 the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-151/

Copy link
Collaborator

@juliodialpad juliodialpad left a comment

Choose a reason for hiding this comment

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

Thanks Brad!

@braddialpad braddialpad merged commit ec96822 into staging Feb 13, 2024
10 of 11 checks passed
@braddialpad braddialpad deleted the fix/font-quotes branch February 13, 2024 17:07
braddialpad pushed a commit that referenced this pull request Feb 13, 2024
## [1.23.6](dialtone-tokens/v1.23.5...dialtone-tokens/v1.23.6) (2024-02-13)

### Bug Fixes

* extra quotes on font tokens ([#151](#151)) ([ec96822](ec96822))
braddialpad pushed a commit that referenced this pull request Feb 13, 2024
# [9.13.0](dialtone/v9.12.0...dialtone/v9.13.0) (2024-02-13)

### Bug Fixes

* **Emoji:** skeleton display issue ([#150](#150)) ([5ccf8ca](5ccf8ca))
* extra quotes on font tokens ([#151](#151)) ([ec96822](ec96822))
* **Feed Item Row:** change typography to compact ([#149](#149)) ([47f4a45](47f4a45))
* load archivo font ([#152](#152)) ([3147749](3147749))
* rich editor improvements ([#142](#142)) ([a776d47](a776d47))

### Documentation

* **Typography:** design language ([#75](#75)) ([6d6ecd5](6d6ecd5))

### Features

* new icon user speak ([#147](#147)) ([b8a9ac9](b8a9ac9))
* text decoration scopes ([#148](#148)) ([5e58ef1](5e58ef1))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-visual-test Add this tag when the PR does not need visual testing
Projects
None yet
2 participants