[GH-ISSUE #1621] [Bug]: Mobile: Account ledger jumps on first selection #7550

Closed
opened 2026-04-10 17:25:49 -05:00 by GiteaMirror · 13 comments
Owner

Originally created by @shall0pass on GitHub (Aug 31, 2023).
Original GitHub issue: https://github.com/actualbudget/actual/issues/1621

Verified issue does not already exist?

  • I have searched and found no existing issue

What happened?

If I scroll down an account ledger on the mobile interface and select a transaction, the first time I try the interface will jump instead of opening the transaction. The second attempt is successful. The video below only shows the first attempt on the demo budget.

https://github.com/actualbudget/actual/assets/20625555/13f0fd77-ca41-40e1-93a5-495f917547c2

What error did you receive?

No response

Where are you hosting Actual?

Other

What browsers are you seeing the problem on?

Chrome

Operating System

Mobile Device

Originally created by @shall0pass on GitHub (Aug 31, 2023). Original GitHub issue: https://github.com/actualbudget/actual/issues/1621 ### Verified issue does not already exist? - [X] I have searched and found no existing issue ### What happened? If I scroll down an account ledger on the mobile interface and select a transaction, the first time I try the interface will jump instead of opening the transaction. The second attempt is successful. The video below only shows the first attempt on the demo budget. https://github.com/actualbudget/actual/assets/20625555/13f0fd77-ca41-40e1-93a5-495f917547c2 ### What error did you receive? _No response_ ### Where are you hosting Actual? Other ### What browsers are you seeing the problem on? Chrome ### Operating System Mobile Device
GiteaMirror added the bug label 2026-04-10 17:25:49 -05:00
Author
Owner

@Kidglove57 commented on GitHub (Aug 31, 2023):

I am running a recent Edge version on iOS 16. For me, selected transactions are opening immediately. I attach a brief screen recording.
https://github.com/actualbudget/actual/assets/93882449/d68e230b-cde8-40f9-884c-8685f9a305c7

PS this is my first attempt at a screen recording so forgive the quality!

<!-- gh-comment-id:1700873892 --> @Kidglove57 commented on GitHub (Aug 31, 2023): I am running a recent Edge version on iOS 16. For me, selected transactions are opening immediately. I attach a brief screen recording. https://github.com/actualbudget/actual/assets/93882449/d68e230b-cde8-40f9-884c-8685f9a305c7 PS this is my first attempt at a screen recording so forgive the quality!
Author
Owner

@shall0pass commented on GitHub (Aug 31, 2023):

Hmm. I see it with both Brave and Firefox on Android.

<!-- gh-comment-id:1700896024 --> @shall0pass commented on GitHub (Aug 31, 2023): Hmm. I see it with both Brave and Firefox on Android.
Author
Owner

@Kidglove57 commented on GitHub (Aug 31, 2023):

Sorry to be the "spoilsport" here! I have tested on iOS in Safari, Chrome, Brave and Firefox but have not been able to replicate this.

My "Edge" version is about 10 days old - if that makes any difference?

<!-- gh-comment-id:1700937741 --> @Kidglove57 commented on GitHub (Aug 31, 2023): Sorry to be the "spoilsport" here! I have tested on iOS in Safari, Chrome, Brave and Firefox but have not been able to replicate this. My "Edge" version is about 10 days old - if that makes any difference?
Author
Owner

@shall0pass commented on GitHub (Aug 31, 2023):

😄 could you try on the demo budget online? My Edge version is only 2 days old and my video was on the demo version which is current with the master branch.

<!-- gh-comment-id:1700959901 --> @shall0pass commented on GitHub (Aug 31, 2023): 😄 could you try on the demo budget online? My Edge version is only 2 days old and my video was on the demo version which is current with the master branch.
Author
Owner

@shall0pass commented on GitHub (Aug 31, 2023):

Maybe it's a recently introduced bug. I can replicate this on Windows 10 too.
jumping_account_view

<!-- gh-comment-id:1701006477 --> @shall0pass commented on GitHub (Aug 31, 2023): Maybe it's a recently introduced bug. I can replicate this on Windows 10 too. ![jumping_account_view](https://github.com/actualbudget/actual/assets/20625555/d10b524c-4dfa-496c-bd7f-6cb079fce31f)
Author
Owner

@Kidglove57 commented on GitHub (Aug 31, 2023):

Weirder and weirder! The attached is using the online Demo on Mac OS, in Brave. I have exactly the same result using Brave in iOS 16. All working perfectly.

https://github.com/actualbudget/actual/assets/93882449/f5c207c7-a655-48ab-be2b-8a94ec668f72

<!-- gh-comment-id:1701039362 --> @Kidglove57 commented on GitHub (Aug 31, 2023): Weirder and weirder! The attached is using the online Demo on Mac OS, in Brave. I have exactly the same result using Brave in iOS 16. All working perfectly. https://github.com/actualbudget/actual/assets/93882449/f5c207c7-a655-48ab-be2b-8a94ec668f72
Author
Owner

@shall0pass commented on GitHub (Aug 31, 2023):

Odd. Try scrolling down a bit first.

<!-- gh-comment-id:1701056960 --> @shall0pass commented on GitHub (Aug 31, 2023): Odd. Try scrolling down a bit first.
Author
Owner

@Kidglove57 commented on GitHub (Aug 31, 2023):

This is interesting:

  1. I am still unable to reproduce this in my production file.
  2. Using the Demo file - and scrolling down further as requested - I am now sometimes able to reproduce what you are seeing. But not consistently.

If I do reproduce this it is only on the first click in the transaction register and all works fine after that. It feels just like it would if I did not clicked cleanly in the required place. Like I had "miss hit". Without your perseverance I think I would just have passed it off as a passing glitch.

<!-- gh-comment-id:1701082212 --> @Kidglove57 commented on GitHub (Aug 31, 2023): This is interesting: 1) I am still unable to reproduce this in my production file. 2) Using the Demo file - and scrolling down further as requested - I am now sometimes able to reproduce what you are seeing. But not consistently. If I do reproduce this it is only on the first click in the transaction register and all works fine after that. It feels just like it would if I did not clicked cleanly in the required place. Like I had "miss hit". Without your perseverance I think I would just have passed it off as a passing glitch.
Author
Owner

@joel-jeremy commented on GitHub (Aug 31, 2023):

I am able to reproduce this on the demo instance.

https://github.com/actualbudget/actual/assets/20313680/2d66f446-e9f4-45d0-8800-0b93a29c6fb9

<!-- gh-comment-id:1701135913 --> @joel-jeremy commented on GitHub (Aug 31, 2023): I am able to reproduce this on the demo instance. https://github.com/actualbudget/actual/assets/20313680/2d66f446-e9f4-45d0-8800-0b93a29c6fb9
Author
Owner

@MatissJanis commented on GitHub (Aug 31, 2023):

Repro:

  1. open a single account on mobile
  2. scroll down
  3. click BETWEEN transactions on the right side (so right of the date)
  4. scroll will jump to top

I am not sure what's going on here.. but good catch!

<!-- gh-comment-id:1701734608 --> @MatissJanis commented on GitHub (Aug 31, 2023): Repro: 1. open a single account on mobile 2. scroll down 3. click BETWEEN transactions on the right side (so right of the date) 4. scroll will jump to top I am not sure what's going on here.. but good catch!
Author
Owner

@trevdor commented on GitHub (Sep 1, 2023):

Turns out the first time you interact with the list, react-aria sets the focused element to the first list item, then scrolls that focused element into view.
I think this was a bug, and the relevant code has changed quite a lot in the latest version of useSelectableList--the offending onFocus handler isn't even there anymore.

Appears fixed by upgrading. I went ahead and upgrade all react-aria packages, as they get frequent updates and work with one another. I'll do some thorough manual testing of the mobile components that use these libs.

<!-- gh-comment-id:1702024623 --> @trevdor commented on GitHub (Sep 1, 2023): Turns out the first time you interact with the list, react-aria sets the focused element to the first list item, then scrolls that focused element into view. I think this was a bug, and the relevant code has changed quite a lot in the latest version of `useSelectableList`--the offending `onFocus` handler isn't even there anymore. Appears fixed by upgrading. I went ahead and upgrade all react-aria packages, as they get frequent updates and work with one another. I'll do some thorough manual testing of the mobile components that use these libs.
Author
Owner

@Kidglove57 commented on GitHub (Sep 1, 2023):

It’s weird though how I can only reproduce this in the demo.

My own personal file behaves perfectly on mobile and selected transactions open immediately.

<!-- gh-comment-id:1702536679 --> @Kidglove57 commented on GitHub (Sep 1, 2023): It’s weird though how I can only reproduce this in the demo. My own personal file behaves perfectly on mobile and selected transactions open immediately.
Author
Owner

@shall0pass commented on GitHub (Sep 1, 2023):

@trevdor It's amazing to me that you were able to track this down so quickly! Well done! Your PR does indeed fix the jumping I was experiencing.

Thanks!

<!-- gh-comment-id:1702682064 --> @shall0pass commented on GitHub (Sep 1, 2023): @trevdor It's amazing to me that you were able to track this down so quickly! Well done! Your PR does indeed fix the jumping I was experiencing. Thanks!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#7550