Commit 3dcd5c8
Partial Mitigation for Samsung TextInput Hangs (#35967)
Summary:
Pull Request resolved: #35967
In #35936 we observed that the presence of AbsoluteSizeSpan may lead to hangs when using the Grammarly keyboard on Samsung.
This mitigation makes it so that we do not emit this span in the most common cases, when it is sufficient to set `android:textSize`. In simple cases, it causes typing into the TextInput to no longer hang.
This does not resolve the issue for TextInputs which meaningfully use layout-effecting spans (or at least font size), such as non-uniform text size within the input. We could potentially do further work to reduce the number of spans emitted in these scenarios, but this may be fighting a losing battle against the platform.
Changelog:
[Android][Fixed] - Partial Mitigation for Samsung TextInput Hangs (Paper)
Reviewed By: cortinico
Differential Revision: D42721684
fbshipit-source-id: 1b6935014c8055c3bd547c7ba5294bb9975479c21 parent a8166bd commit 3dcd5c8
File tree
1 file changed
+29
-0
lines changed- ReactAndroid/src/main/java/com/facebook/react/views/textinput
1 file changed
+29
-0
lines changedLines changed: 29 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
585 | 585 | | |
586 | 586 | | |
587 | 587 | | |
| 588 | + | |
| 589 | + | |
| 590 | + | |
| 591 | + | |
588 | 592 | | |
589 | 593 | | |
590 | 594 | | |
| |||
658 | 662 | | |
659 | 663 | | |
660 | 664 | | |
| 665 | + | |
| 666 | + | |
| 667 | + | |
| 668 | + | |
| 669 | + | |
| 670 | + | |
| 671 | + | |
| 672 | + | |
| 673 | + | |
| 674 | + | |
| 675 | + | |
| 676 | + | |
| 677 | + | |
| 678 | + | |
| 679 | + | |
| 680 | + | |
| 681 | + | |
| 682 | + | |
| 683 | + | |
| 684 | + | |
| 685 | + | |
| 686 | + | |
| 687 | + | |
| 688 | + | |
| 689 | + | |
661 | 690 | | |
662 | 691 | | |
663 | 692 | | |
| |||
0 commit comments