refactor: [M3-8651] - Migrate Tabs
to ui
package
#11320
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description 📝
Move
Tabs
and associated components, story and tests toui
packageChanges 🔄
Tabs
and associated components (SafeTabPanel
,TabLinkList
,TabList
,TabPanel
,TabPanels
,Tab
), story and tests to@linode/ui
packageMemoryRouter
to ui package'stestHelpers
utilitiesOptions
typeTab
type toTabProps
to avoid naming conflicts with theTab
component in the barrelindex.ts
file and other parts of the manager package.Target release date 🗓️
N/A
How to test 🧪
Tabs
is used@linode/ui
in@linode/ui
rm -r packages/manager/node_modules/.cache
+ clear your browser cache before testing StorybookAs an Author, I have considered 🤔
As an Author, before moving this PR from Draft to Open, I confirmed ✅