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(lint): NO-JIRA fix linter rule for text styles #494

Merged
merged 2 commits into from
Sep 11, 2024

Conversation

ninamarina
Copy link
Contributor

@ninamarina ninamarina commented Sep 10, 2024

Fix recommend-typography-style rule false positives

In this rule we were checking for d-fw and d-ff prefixes, which are also used for flex-wrap and flex-flow properties.
I improved the rule to target the specific properties to avoid getting a warning in those cases.

image

@ninamarina ninamarina marked this pull request as ready for review September 10, 2024 15:28
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.

That's something we should fix in the near future, it's not ok that more that one utility class has the same prefix, can lead to confusions like the one you just had with d-fw-unset

Just apply the suggested change before merging, thanks!

…-style.js

Co-authored-by: Julio Ortega <julio.ortega@dialpad.com>
Copy link

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

@ninamarina ninamarina merged commit c6b5f3f into staging Sep 11, 2024
8 checks passed
@ninamarina ninamarina deleted the fix/linter-rule branch September 11, 2024 17:44
juliodialpad pushed a commit that referenced this pull request Sep 11, 2024
## [9.72.3](dialtone/v9.72.2...dialtone/v9.72.3) (2024-09-11)

### Bug Fixes

* **Lint:** NO-JIRA fix linter rule for text styles ([#494](#494)) ([c6b5f3f](c6b5f3f))
* **Scrollbar Directive:** NO-JIRA overlayscrollbars missing styles ([#493](#493)) ([d1a88e1](d1a88e1))
* **Tokens:** NO-JIRA keep references in css output ([#487](#487)) ([b019ff6](b019ff6))

### Documentation

* **Icon:** DLT-1916 update documentation to show tree shaking examples   ([#491](#491)) ([5b9d969](5b9d969))
juliodialpad pushed a commit that referenced this pull request Sep 11, 2024
## [1.7.2](eslint-plugin-dialtone/v1.7.1...eslint-plugin-dialtone/v1.7.2) (2024-09-11)

### Bug Fixes

* **Lint:** NO-JIRA fix linter rule for text styles ([#494](#494)) ([c6b5f3f](c6b5f3f))
juliodialpad pushed a commit that referenced this pull request Sep 12, 2024
# [9.73.0-beta.2](dialtone/v9.73.0-beta.1...dialtone/v9.73.0-beta.2) (2024-09-12)

### Bug Fixes

* **Lint:** NO-JIRA fix linter rule for text styles ([#494](#494)) ([c6b5f3f](c6b5f3f))
* **Scrollbar Directive:** NO-JIRA overlayscrollbars missing styles ([#493](#493)) ([d1a88e1](d1a88e1))
* **Tokens:** NO-JIRA keep references in css output ([#487](#487)) ([b019ff6](b019ff6))

### Documentation

* **Icon:** DLT-1916 update documentation to show tree shaking examples   ([#491](#491)) ([5b9d969](5b9d969))
* NO-JIRA wrong dialtone-css import on local ([da5f7d6](da5f7d6))

### Features

* **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))
* **Message Input:** DLT-1916 remove dt-icon from message input ([#490](#490)) ([3215116](3215116))
* **Utilities:** DLT-1763 add gap utility class ([#492](#492)) ([835d06f](835d06f))
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