You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ICU provides two functions which are relevant here: uloc_getCharacterOrientation and uloc_isRightToLeft.
The "characterOrder" property can be retrieved through uloc_getCharacterOrientation, but the V8 implementation uses uloc_isRightToLeft for its CharacterDirectionOfLocale implementation. Both ICU functions can produce different results when non-default scripts are used. For example for "en-Arab", uloc_getCharacterOrientation returns ULOC_LAYOUT_LTR, whereas uloc_isRightToLeft returns true.
The spec should be written so that implementers know which ICU function has to be called. I think it's probably best to remove "characterOrder" from CharacterDirectionOfLocale, because it seems to hint into the wrong direction (uloc_getCharacterOrientation, but we actually want uloc_isRightToLeft, right?).
The text was updated successfully, but these errors were encountered:
CharacterDirectionOfLocale references "characterOrder":
"characterOrder" is defined in UTS 35 here: https://unicode.org/reports/tr35/tr35-general.html#Layout_Elements.
ICU provides two functions which are relevant here:
uloc_getCharacterOrientation
anduloc_isRightToLeft
.The "characterOrder" property can be retrieved through
uloc_getCharacterOrientation
, but the V8 implementation usesuloc_isRightToLeft
for itsCharacterDirectionOfLocale
implementation. Both ICU functions can produce different results when non-default scripts are used. For example for"en-Arab"
,uloc_getCharacterOrientation
returnsULOC_LAYOUT_LTR
, whereasuloc_isRightToLeft
returnstrue
.The spec should be written so that implementers know which ICU function has to be called. I think it's probably best to remove "characterOrder" from
CharacterDirectionOfLocale
, because it seems to hint into the wrong direction (uloc_getCharacterOrientation
, but we actually wantuloc_isRightToLeft
, right?).The text was updated successfully, but these errors were encountered: