redux-form/redux-form

[React Native] composite character issue (no support for Korean)

Open

#3,938 opened on 2018年3月25日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)JavaScript (12,592 stars) (1,683 forks)batch import
Version 7bughelp wanted

説明

Are you submitting a bug report or a feature request?

I found a bug with ReduxForm

What is the current behavior?

This bug still happens when I put Korean characters. breaks character composition.

What is the expected behavior?

(Deck Title : 가ㄱㅏㄱㅏ should be 가가가) In korean, to input charater '가', I have to enter 'ㄱ' and then 'ㅏ', and they are composed into '가' But, when value changed to non-empty value from empty value, it breaks a composition. So I can't enter '가' to the TextField

What's your environment?

macOs 10.13.3 "react": "16.2.0", "react-native": "0.52.0", "redux": "^3.7.2", "redux-form": "^7.3.0"

Other information

It looks ReduxForm has a bug.
because normal TextInput element or tcomb-native works well. I think ReduxForm re-rendering is breaking the composition.

コントリビューターガイド

技術スタック
reactreact nativejavascript
領域
frontendmobile
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Basic understanding of React NativeFamiliarity with ReduxKnowledge of redux form libraryUnderstanding of composite characters in IME
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Investigate how redux form's Field component and its onChange handler interact with React Native's TextInput during IME composition. The issue suggests that re rendering triggered by redux form breaks the composition. Look at the source code in src/ tests /Field.spec.js and src/createFieldProps.js to understand the event handling. Consider if debouncing the onChange or checking for isComposing event property could prevent the interference. Also review React Native's TextInput documentation on composition events.