-
Notifications
You must be signed in to change notification settings - Fork 0
Add number fields and dist #3
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
hmmm. Seems this will increase the bundle size quite a bit...
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.
Thanks for calling that out.
For our ember bundles, we pay as you go since we import only the locale data that's targeted. Not terribly concerned.
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.
@jasonmit Yeah that is a good point! The only thing I noticed is we are duplicating
pluralRuleFunction
for a particular lang. One in the vendor dist and the other in the app bundle.I'm not 100% sure the locale data for this lib is needed in the vendor.js file, but perhaps you know much better than I do. My understanding was that at runtime, we handoff the relevant data generated from broccoli-cldr-data to
IntlMessageFormat
. Don't see usrequire
ing this data fromintl-messageformat
anywhere. Lmk your thoughts!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.
Sounds like a bug, this is happening on master & easy to reproduce? If so, will investigate soon.
I could see where this could be the case for
en
but would not expect it for any other locales (lib defaults https://github.com/yahoo/intl-messageformat/blob/master/dist/intl-messageformat.js#L1921 https://github.com/yahoo/intl-relativeformat/blob/master/dist/intl-relativeformat.js#L1879)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.
Oh you are right! Other languages does not add anything to the vendor output. 👍