Skip to content

[Android] do not call setHyphenationFrequency on AndroidSdk < 23 - #29258

Closed
fabOnReact wants to merge 1 commit into
react:masterfrom
fabOnReact:hyphenation-missing-method
Closed

[Android] do not call setHyphenationFrequency on AndroidSdk < 23#29258
fabOnReact wants to merge 1 commit into
react:masterfrom
fabOnReact:hyphenation-missing-method

Conversation

@fabOnReact

@fabOnReactfabOnReact commented Jul 1, 2020

Copy link
Copy Markdown
Contributor

Summary

@JoshuaGross This issue fixes#28279 as discussed in #29157 (comment)
Avoid calling setHyphenationFrequency on Android Sdk < 23.

Changelog

[Android] [Fixed] - do not call setHyphenationFrequency on AndroidSdk < 23

Test Plan

BEFOREAFTER

The warning displayed with adb logcat | grep -P "ReactTextAnchorViewManager"

image

I remain available to do improvements. Thanks a lot. Fabrizio.

@facebook-github-botfacebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 1, 2020
@react-native-botreact-native-bot added Bug Platform: Android Android applications. labels Jul 1, 2020
@analysis-bot

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
ios-universaln/a--

Base commit: e261f02

@analysis-bot

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a6,763,081146
androidhermesarmeabi-v7a6,426,157160
androidhermesx867,150,839164
androidhermesx86_647,040,770156
androidjscarm64-v8a8,935,434149
androidjscarmeabi-v7a8,590,898154
androidjscx868,766,294143
androidjscx86_649,341,854150

Base commit: e261f02

@facebook-github-botfacebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@mdvacca has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mdvacca

Copy link
Copy Markdown
Contributor

@fabriziobertoglio1987 can you update the documentation with this change as well?

1 similar comment
@mdvacca

Copy link
Copy Markdown
Contributor

@fabriziobertoglio1987 can you update the documentation with this change as well?

@mdvaccamdvacca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me!

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @fabriziobertoglio1987 in 7d8aeb4.

When will my fix make it into a release? | Upcoming Releases

@fabOnReact

fabOnReact commented Jul 3, 2020

Copy link
Copy Markdown
ContributorAuthor

Thanks a lot! react/react-native-website#2007

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BugCLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.MergedThis PR has been merged.Platform: AndroidAndroid applications.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Android ignores unicode soft hyphen (u00AD)

5 participants

@fabOnReact@analysis-bot@mdvacca@react-native-bot@facebook-github-bot