Fix onSelectionChange event called twice with different payload on iOS for multiline

[tomekzaw]

See RCA in #185 (comment)

Details

Screen.Recording.2024-02-15.at.10.57.28.mov

Related Issues

Fixes #185.

Manual Tests

Linked PRs

[robertKozik]

Changes look good - there is no more excess event on my end.

I saw that you pinpoint the root cause to the swizzled updeteLocalData method - as I recall we implemented it to have correct attributed string during the string measurement. Maybe it's worth to consider finding another workaround to get RCTBaseTextInputShadowView::measurableAttributedText work correctly with markdown attributedString as the enhancment follow-up? What do you think?

[tomekzaw]

@perunt confirmed that this PR fixes the reported issue: https://expensify.slack.com/archives/C01GTK53T8Q/p1708595074853919?thread_ts=1707846163.324679&cid=C01GTK53T8Q

[tomekzaw]

Maybe it's worth to consider finding another workaround to get RCTBaseTextInputShadowView::measurableAttributedText work correctly with markdown attributedString as the enhancment follow-up?

Yes, that's definitely worth exploring. Let's merge this PR in its current shape now and investigate this topic soon.