Skip to content
This repository has been archived by the owner on Oct 19, 2023. It is now read-only.

Add Bidi utilities #42

Merged
merged 50 commits into from
Jul 30, 2018
Merged

Add Bidi utilities #42

merged 50 commits into from
Jul 30, 2018

Conversation

ianegordon
Copy link
Contributor

This includes categories on NSString and NSLocale to assist clients in properly wrapping string with unicode bidi control characters to allow LTR and RTL script in the same string.

@ianegordon ianegordon requested a review from Arcank as a code owner July 26, 2018 04:07
@ianegordon ianegordon removed the request for review from Arcank July 26, 2018 04:19
@ianegordon ianegordon requested a review from jverkoey July 30, 2018 18:58
Copy link
Contributor

@jverkoey jverkoey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it!

+ (BOOL)mdf_isDefaultLanguageRTL {
NSString *languageCode = [[NSLocale currentLocale] objectForKey:NSLocaleLanguageCode];
NSLocaleLanguageDirection characterDirection =
[NSLocale characterDirectionForLanguage:languageCode];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[stylenit] Indentation here and above.

NSLocaleLanguageDirectionUnknown. If NSLocaleLanguageDirectionUnknown, the direction of the string
will be calculated with mdf_calculatedLanguageDirection.

|contextDirection| must be specificed and cannot be unknown. Only
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[typo] specified

@ianegordon ianegordon merged commit 340899b into develop Jul 30, 2018
@ianegordon ianegordon deleted the experimental-bidi branch October 2, 2018 12:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants