Moment: Module import doesn't work in the latest Chrome

Created on 5 Oct 2017 · 7Comments · Source: moment/moment

Description of the Issue and Steps to Reproduce:

Latest version of Chrome declares native modules import support. But the feature requires full module file name to be provided (including ".js" extension).
In the library code "import ..." entries don't have ".js" extensions which results 404 errors in the browser.

FYI: One thing that might (and does on some platforms) hide this issue is a web-server configuration.
Apache web-server on OSX seems has "MultiViews" setting enabled by default (which makes the server find "moment.js" file if there is no "moment" file found), but on Ubuntu, for example, this web-server option is disabled which make the module loading fail.

Environment:

Chrome 61.0.3163.100 (64 bit) on Ubuntu 16.04

console.log( (new Date()).toString())
console.log((new Date()).toLocaleString())
console.log( (new Date()).getTimezoneOffset())
console.log( navigator.userAgent)
VM3015:1 Thu Oct 05 2017 19:30:43 GMT+0600 (+06)
VM3015:2 05.10.2017, 19:30:43
VM3015:3 -360
VM3015:4 Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36

moment.js version is 2.18.1

Help Wanted Loading

Source

arcady-zherdev

👍6

Most helpful comment

This is not just a chrome thing. As far as i know, the es6 module spec requires a file name in the FromClause and not just a string. Rollup and Webpack are working fine without the file extension, but that's only to make bundling easier.

And overall it's better to avoid letting any loader guess the extension, so yeah, the FromClauses should really be full file names.

nnmrts on 8 Jan 2018

👍3

All 7 comments

And overall it's better to avoid letting any loader guess the extension, so yeah, the FromClauses should really be full file names.

nnmrts on 8 Jan 2018

👍3

Would be really good to get this sorted as neither the source version nor the compiled version works when using a module import in browsers which support it.

It _should_ be a quick fix of just adding file extensions to the imports.
Also for people wanting to use the compiled version - for global use of Moment, the global property needs to fallback to window as this is undefined.