io-app icon indicating copy to clipboard operation
io-app copied to clipboard

Input lag when pressing buttons on Android

Open bfabio opened this issue 5 years ago • 8 comments
trafficstars

There's a noticeable input lag when pressing buttons on Android with every device I have. Granted they're not the fastest out there, but neither will be the average device where IO will be installed, besides, it happens on the emulator as well.

If I had to eyeball it, I wanna say there's about a 100ms delay between the touch and the ripple animation which makes the app look sluggish and not reactive. I think this is due to crossing the React Native bridge.

This is eb86a36b running on the emulator: https://imgur.com/a/8K2gZzk

and this is a patch using BaseButton from react-native-gesture-handler, which doesn't cross the bridge. Please ignore the lack of styling for now: https://imgur.com/a/ZBvwLdG

It makes a world of difference in how the app's performance is perceived.

Smartphone

  • Device: One Plus One, Xiaomi Mi A1, Emulator
  • OS: Android 9, Android 10

bfabio avatar May 15 '20 19:05 bfabio

This issue is stale because it has been open for 60 days with no activity.

github-actions[bot] avatar Sep 08 '21 02:09 github-actions[bot]

This issue was closed because it has been inactive for 14 days since being marked as stale.

github-actions[bot] avatar Sep 22 '21 02:09 github-actions[bot]

@pietro909 @fabriziofff I'm not sure autoclosing stale issues after a while is a good idea. This one is still valid.

bfabio avatar Sep 26 '21 12:09 bfabio

The autoclosure occurs only if there are no further comments from the reporter or anyone else within the next 14 days, after being marked as a stale. In case the issue is still valid we can reopen it without problem!

fabriziofff avatar Sep 29 '21 14:09 fabriziofff

@fabriziofff thanks! :)

bfabio avatar Sep 29 '21 17:09 bfabio

@bfabio first off, thank you for your valuable report and effort. I understand that keeping this issue in line for some ~18 months must feel frustrating.

With that being said, I kindly ask you please to confirm whether the issue is still valid with the last configuration you reported and the latest production tag 1.36.0-rc.2:

Device: One Plus One, Xiaomi Mi A1, Emulator
OS: Android 9, Android 10

We'll then use this newer information to triage the issue internally. Thanks!

pietro909 avatar Nov 04 '21 08:11 pietro909

@pietro909 don't worry, I know the changes required can touch a lot of code and most likely need to be postponed to a quieter time :)

I'm perfectly fine with just having the issue tracked and acknowledge IO strives for great UX experience - which it already does (who knows, someone might even pick up the work)

I can confirm the issue is still valid for the Emulator and my One Plus One. I can't confirm with my Mi A1 because I dropped it down a flight of stairs a few months ago :(

bfabio avatar Nov 30 '21 12:11 bfabio

Tracked via https://pagopa.atlassian.net/browse/IA-632

pietro909 avatar Jan 17 '22 16:01 pietro909