-
Notifications
You must be signed in to change notification settings - Fork 238
chore: Deprecated old theming utilities #3519
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
chore: Deprecated old theming utilities #3519
Conversation
Workday/canvas-kit
|
Project |
Workday/canvas-kit
|
Branch Review |
mc-deprecate-theming-utils
|
Run status |
|
Run duration | 02m 53s |
Commit |
|
Committer | Manuel Carrera |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
1
|
|
86
|
|
0
|
|
843
|
View all changes introduced in this branch ↗︎ |
UI Coverage
19.41%
|
|
---|---|
|
1526
|
|
365
|
Accessibility
99.31%
|
|
---|---|
|
6 critical
5 serious
0 moderate
2 minor
|
|
77
|
import {memoize} from '../utils/memoize'; | ||
|
||
/** | ||
* @deprecated ⚠️ `shiftColor` is deprecated and will be removed in a future major version. While we work on a algorithm for color shifting, you can use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: You can use what?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made some nits, but otherwise this is good. Thanks, Manny!
} | ||
|
||
/** | ||
* @deprecated ⚠️ `styled` is deprecated and will be removed in a future major version. Please use `createStyles` or `createStencil` instead with [CSS logical properties](https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_logical_properties_and_values). For more information, view our [Styling docs](https://workday.github.io/canvas-kit/?path=/docs/styling-getting-started-overview--docs). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YES. FI-NA-LLY. It's happening.
Co-authored-by: Alan B Smith <[email protected]>
import {memoize} from '../utils/memoize'; | ||
|
||
/** | ||
* @deprecated ⚠️ `shiftColor` is deprecated and will be removed in a future major version. While we work on a algorithm for color shifting, you can use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* @deprecated ⚠️ `shiftColor` is deprecated and will be removed in a future major version. While we work on a algorithm for color shifting, you can use | |
* @deprecated ⚠️ `shiftColor` is deprecated and will be removed in a future major version. While we work on a algorithm for color shifting, you can use [oklch with calc or from](https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/oklch) to choose colors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary
Fixes: #3518
Release Category
Utilities
Release Note
The following theming utils have been deprecated:
defaultCanvasTheme
styled
createCanvasTheme
getTheme
useTheme
useIsRTL
useThemedRing
To better support our theming we encourage our consumers to use our CSS Tokens. For more information, please view our Theming Docs and CSS Token docs.
Checklist
ready for review
has been added to PRFor the Reviewer
Where Should the Reviewer Start?
Areas for Feedback? (optional)
Testing Manually
Screenshots or GIFs (if applicable)
Thank You Gif (optional)