-
-
Notifications
You must be signed in to change notification settings - Fork 32.6k
[react-transition-group] Remove the package from the repo #48644
Copy link
Copy link
Open
Labels
breaking changeIntroduces changes that are not backward compatible.Introduces changes that are not backward compatible.scope: code-infraInvolves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).
Milestone
Metadata
Metadata
Assignees
Labels
breaking changeIntroduces changes that are not backward compatible.Introduces changes that are not backward compatible.scope: code-infraInvolves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).
Fields
Give feedbackNo fields configured for issues without a type.
Projects
StatusShow more project fields
To process
Steps to reproduce
Steps:
Current behavior
react-transition-group is still in the repo, although it's not useful anymore, as we migrated away from it using an in-house solution here.
Expected behavior
react-transition-group should be removed
Context
react-transition-group was introduced here - #48325, doesn't have proper exports map in its package.json leading me to have a workaround in X.
IMO, we should remove this package altogether since it hasn't been updated since 4 years. It wasn't used anyways before the PR.
I was looking into removing it last week as it's causing unnecessary bundle bloat as well. It seems like it's only here to do a context reset. I guess we could maybe move this to a userland solution through Slots?
Your environment
No response
Search keywords: react-transition-group