-
Notifications
You must be signed in to change notification settings - Fork 109
feat(many): remove deprecated v2 items #2549
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -519,6 +519,10 @@ type: embed | |
|
|
||
| ``` | ||
|
|
||
| ### DrawerLayout | ||
|
|
||
| - Previously deprecated `_content` prop has been removed from `DrawerLayout.Tray` and `DrawerLayout.Content`. Use `ref` instead. | ||
|
|
||
| ### Drilldown | ||
|
|
||
| Drilldown itself has no token changes. However, its sub-components share tokens with Options components, so the token changes listed under Options apply here as well: | ||
|
|
@@ -581,6 +585,8 @@ type: embed | |
|
|
||
| ### FormField | ||
|
|
||
| - Previously deprecated `FormFieldLabel` has been removed. | ||
|
|
||
| #### FormFieldGroup | ||
|
|
||
| `error` or `success` messages are no longer displayed when the component is `readOnly` or `disabled`. | ||
|
|
@@ -709,6 +715,8 @@ type: embed | |
|
|
||
| The default value of the `color` prop has changed from `'inherit'` to `'primary'`. If you need to preserve the previous behavior, explicitly set `color="inherit"`. | ||
|
|
||
| - Previously deprecated `'ai'` value of the `color` prop has been removed. Use the `aiVariant` prop instead. | ||
|
|
||
| ```js | ||
| --- | ||
| type: embed | ||
|
|
@@ -742,6 +750,8 @@ type: embed | |
| ### Link | ||
|
|
||
| - `isWithinText` prop has been removed. | ||
| - Previously deprecated `_link` prop has been removed. Use `ref` instead. | ||
| - `'inline-small'` and `'standalone-small'` values of the `variant` prop have been removed. Use `variant="inline"` or `variant="standalone"` with `size="small"` instead. | ||
|
|
||
| #### New `size` prop | ||
|
|
||
|
|
@@ -751,13 +761,6 @@ A new `size` prop has been added to control the font size, line height, and icon | |
| - `'medium'` (default) | ||
| - `'large'` | ||
|
|
||
| #### Deprecated variant values | ||
|
|
||
| The following variant values have been **deprecated** and will be removed in a future version (still supported but warn): | ||
|
|
||
| - `'inline-small'` | ||
| - `'standalone-small'` | ||
|
|
||
| ```js | ||
| --- | ||
| type: embed | ||
|
|
@@ -811,6 +814,8 @@ type: embed | |
|
|
||
| #### Menu.Item | ||
|
|
||
| - Previously deprecated `_node` prop has been removed. Use `ref` instead. | ||
|
|
||
| ```js | ||
| --- | ||
| type: embed | ||
|
|
@@ -1029,6 +1034,10 @@ type: embed | |
|
|
||
| ``` | ||
|
|
||
| ### Pagination | ||
|
|
||
| - Previously deprecated `_root` prop has been removed. Use `ref` instead. | ||
|
|
||
| ### Pill | ||
|
|
||
| #### Deprecated color prop values | ||
|
|
@@ -1189,7 +1198,10 @@ type: embed | |
| <V12ChangelogTable | ||
| removed={[ | ||
| {name:"handleFocusRingSize",note:"style uses sharedTokens.focusOutline.width token"}, | ||
| {name:"handleFocusRingColor",note:"style uses sharedTokens.focusOutline.onColor token"} | ||
| {name:"handleFocusRingColor",note:"style uses sharedTokens.focusOutline.onColor token"}, | ||
| {name:"handleFocusOutlineColor",note:"only used by the removed deprecated thumb variant"}, | ||
| {name:"handleFocusOutlineWidth",note:"only used by the removed deprecated thumb variant"}, | ||
| {name:"handleShadowColor",note:"only used by the removed deprecated thumb variant"} | ||
|
Comment on lines
+1202
to
+1204
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As the thumb variant was removed here #2544 but I realized these tokens should be removed as well so I updated this and told the designers to remove these tokens too. |
||
| ]} | ||
| changed={[ | ||
| {oldName:"handleShadow",newName:"boxShadow",note:""}, | ||
|
|
@@ -1256,6 +1268,10 @@ type: embed | |
|
|
||
| ``` | ||
|
|
||
| ### SimpleSelect | ||
|
|
||
| - Previously deprecated `_select` prop has been removed. Use `ref` instead. | ||
|
|
||
| ### SourceCodeEditor | ||
|
|
||
| ```js | ||
|
|
@@ -1394,6 +1410,8 @@ type: embed | |
|
|
||
| ### Tag | ||
|
|
||
| - Previously deprecated `title` prop has been removed. | ||
|
|
||
| ```js | ||
| --- | ||
| type: embed | ||
|
|
@@ -1486,6 +1504,10 @@ type: embed | |
|
|
||
| ``` | ||
|
|
||
| ### TimeSelect | ||
|
|
||
| - Previously deprecated `_select` prop has been removed. Use `ref` instead. | ||
|
|
||
| ### ToggleDetails | ||
|
|
||
| ```js | ||
|
|
@@ -1527,12 +1549,18 @@ type: embed | |
|
|
||
| ### TopNavBar | ||
|
|
||
| #### TopNavBar.Layout | ||
|
|
||
| - `hideActionsUserSeparator` prop has been removed. | ||
|
Comment on lines
+1553
to
+1554
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this prop was removed in the TopNavrBar rework #2512 because it was a fully 'dead' prop but I realized this was missing from the upgrade guide. |
||
|
|
||
| #### TopNavBar.Brand | ||
|
|
||
| The deprecated `renderName` and `nameBackground` props have been removed. Please remove them from your code. | ||
|
|
||
| ### TreeBrowser | ||
|
|
||
| - Previously deprecated `_root` prop has been removed. Use `ref` instead. | ||
|
|
||
| #### Icon system migration | ||
|
|
||
| TreeBrowser now uses Lucide icons instead of the legacy icon system: | ||
|
|
@@ -1575,8 +1603,15 @@ type: embed | |
|
|
||
| ``` | ||
|
|
||
| ### TruncateText | ||
|
|
||
| - Previously deprecated `_ref` prop has been removed. Use `ref` instead. | ||
|
|
||
| ### View | ||
|
|
||
| - Previously deprecated `_element` prop has been removed. Use `ref` instead. | ||
| - Previously deprecated `focusRingBorderRadius` prop has been removed. Use the `borderRadius` prop instead. | ||
|
|
||
| ```js | ||
| --- | ||
| type: embed | ||
|
|
||
This file was deleted.
This file was deleted.
Uh oh!
There was an error while loading. Please reload this page.
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.
This was added by @git-nandor before the multi-versioning was introduced so I deleted this as I think the deprecation is not necessary and we can jump straight to removing these values.