Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mitigation for Samsung TextInput Hangs #35967

Closed
wants to merge 1 commit into from

Commits on Jan 26, 2023

  1. Mitigation for Samsung TextInput Hangs (facebook#35967)

    Summary:
    Pull Request resolved: facebook#35967
    
    In facebook#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 any case where it is sufficient to rely on already set EditText textSize. In simple cases, tested on two devices, it causes typing into the TextInput to no longer hang.
    
    This does not fully 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 instead just try to reduce to minimum AbsoluteSizeSpan possible.
    
    Testing the first commit was able to resolve hangs in some simpler inputs tested, by me and cortinico.
    
    Changelog:
    [Android][Fixed] - Mitigation for Samsung TextInput Hangs
    
    Reviewed By: cortinico
    
    Differential Revision: D42721684
    
    fbshipit-source-id: 000a8ceb0200abb19a2aa77cdb78276d289c3115
    NickGerleman authored and facebook-github-bot committed Jan 26, 2023
    Configuration menu
    Copy the full SHA
    14adf07 View commit details
    Browse the repository at this point in the history