Page MenuHomePhabricator

MultimediaViewer should not use moment.js for a single line
Closed, ResolvedPublic

Description

Moment.js is quite a large library (70K gzipped) and is used only in a single line of code within MultimediaViewer. This line could be replaced with [Date.toLocaleString](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/toLocaleString)

See T146798, T137775 for related issues.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 513209 had a related patch set uploaded (by Simon04; owner: Simon04):
[mediawiki/extensions/MultimediaViewer@master] Replace moment.js with Date.toLocaleString

https://gerrit.wikimedia.org/r/513209

Change 513209 merged by jenkins-bot:
[mediawiki/extensions/MultimediaViewer@master] Replace moment.js with Date.toLocaleString

https://gerrit.wikimedia.org/r/513209