Skip to content

Conversation

@bbrto21
Copy link

@bbrto21 bbrto21 commented Aug 19, 2021

  • Use composing methods of TextInputModel in preedit events callback.
  • Make Implementations about editing text simpler.
  • Remove unnecessary public methods of TizenInputMethodContext.
  • Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae boram21.bae@samsung.com

* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
Copy link
Member

@bwikbs bwikbs left a comment

Choose a reason for hiding this comment

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

I didn't understand all the changes, but the processing logic seems very intuitive and neat.
Great job!! Thanks👍

@bwikbs
Copy link
Member

bwikbs commented Aug 27, 2021

@flutter-tizen/maintainers Any comments?

@bbrto21 bbrto21 merged commit f0a8b64 into flutter-tizen:flutter-2.2.1-tizen Aug 30, 2021
swift-kim pushed a commit that referenced this pull request Sep 27, 2021
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Nov 14, 2021
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Dec 9, 2021
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Dec 17, 2021
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Feb 7, 2022
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Feb 11, 2022
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request May 12, 2022
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Aug 5, 2022
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
swift-kim pushed a commit that referenced this pull request Sep 1, 2022
* Use composing methods of TextInputModel in preedit events callback.
* Make Implementations about editing text simpler.
* Remove unnecessary public methods of TizenInputMethodContext.
* Fix a bug that didn't reset IMF-context when the selection base changed

Signed-off-by: Boram Bae <boram21.bae@samsung.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants