You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A trivial webpack app that uses should not include moment.js in the bundle. In 3.20.7 and earlier, this was the case.
What is actually happening?
In 3.21.0 and newer, this simple app pulls in moment.js which bloats the bundle by about 40k even though it's unused. It looks like this was introduced by #17816 which made ConfigProvider depend on LocaleProvider which depends on moment. I believe a fix would be to separate ConfigProvider and ConfigConsumer into separate files.
Environment
Info
antd
3.23.5
React
16.10.1
System
OSX 10.14.6
Browser
Chrome 77.0.3865.90 (Official Build) (64-bit)
This bundle size regression is blocking our ability to upgrade past 3.20.7.
The text was updated successfully, but these errors were encountered:
Yep, without being able to read that bug exactly, it looks the same. But it seems to talk a lot about locales. So I'd also like to clarify that the issue isn't just about locales.
Reproduction link
Steps to reproduce
What is expected?
A trivial webpack app that uses should not include moment.js in the bundle. In 3.20.7 and earlier, this was the case.
What is actually happening?
In 3.21.0 and newer, this simple app pulls in moment.js which bloats the bundle by about 40k even though it's unused. It looks like this was introduced by #17816 which made ConfigProvider depend on LocaleProvider which depends on moment. I believe a fix would be to separate ConfigProvider and ConfigConsumer into separate files.
This bundle size regression is blocking our ability to upgrade past 3.20.7.
The text was updated successfully, but these errors were encountered: