angular-timer icon indicating copy to clipboard operation
angular-timer copied to clipboard

It should be handle by the user's application itself

Open FDiskas opened this issue 9 years ago • 8 comments

Yes please remove this as fast as posible

moment.locale(this.language);

https://github.com/siddii/angular-timer/blob/v1.3.4/dist/angular-timer.js#L378

FDiskas avatar Aug 18 '15 20:08 FDiskas

This bug forces my all application to change the locale. You probably did something wrong with fallback language.

FDiskas avatar Aug 18 '15 20:08 FDiskas

Hi @FDiskas,

Could you explain a bit more

This bug forces my all application to change the locale

Have you seen supported languages ? https://github.com/EvanHahn/HumanizeDuration.js#supported-languages Feel free to make a PR to HumanizeDuration to add your language :)

The fallback language is English, for information.

polomarcus avatar Aug 26 '15 20:08 polomarcus

Befor application is running I'm setting the moment.locale(lt); and then your plugin is reseting this to EN locale. Thats wrong :)

FDiskas avatar Aug 27 '15 09:08 FDiskas

I have the same problem with norwegian, which IS supported by humanizeDuration. Norwegian have a different abbreviation in humanizeDuration and moment, this is probably the case for other languages too. You can not trust two libraries to have the same abbreviations. Why do you even set the language of moment? I can not find anywhere in your code where you use any moment function which uses current language.

I find it strange that you do not think it is very bad for an angular module change global settings of a different library. Especially for something as common as moment.

Please remove this.

yacuzo avatar Sep 29 '15 11:09 yacuzo

+1

SBD580 avatar Jan 17 '16 20:01 SBD580

This cost us a lot of time to find out. It should be really removed or at least detect if moment locale is really en (moment.locale())

MartinNuc avatar Feb 26 '16 11:02 MartinNuc

+1 remove it please

Pirozek avatar Mar 15 '16 14:03 Pirozek

+1

mzaarour avatar Aug 02 '16 06:08 mzaarour