Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit 8fa8dde

Browse files
fix(i18n): by default put negative sign before currency symbol
It seems that the case where the negative sign goes between the currency symbol and the numeric value is actually the special case and that locales that require this have it built in. So we should default to having the negative sign before the symbol. See http://cldr.unicode.org/translation/number-patterns and http://unicode.org/cldr/trac/ticket/5674 Closes #10158
1 parent 380ff6f commit 8fa8dde

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/ng/filter/filters.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,9 @@
4545
}
4646
element(by.model('amount')).clear();
4747
element(by.model('amount')).sendKeys('-1234');
48-
expect(element(by.id('currency-default')).getText()).toBe('($1,234.00)');
49-
expect(element(by.id('currency-custom')).getText()).toBe('(USD$1,234.00)');
50-
expect(element(by.id('currency-no-fractions')).getText()).toBe('(USD$1,234)');
48+
expect(element(by.id('currency-default')).getText()).toBe('-$1,234.00');
49+
expect(element(by.id('currency-custom')).getText()).toBe('-USD$1,234.00');
50+
expect(element(by.id('currency-no-fractions')).getText()).toBe('-USD$1,234');
5151
});
5252
</file>
5353
</example>

0 commit comments

Comments
 (0)