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

Mouse wheel scrolling rate is very slow in Split Transactions dialog #5382

Closed
3 of 5 tasks
tactilis opened this issue Nov 25, 2022 · 7 comments
Closed
3 of 5 tasks

Mouse wheel scrolling rate is very slow in Split Transactions dialog #5382

tactilis opened this issue Nov 25, 2022 · 7 comments
Assignees
Milestone

Comments

@tactilis
Copy link

MMEX version:

  • 1.6.1
  • 1.6.2 Alpha

Operating System:

  • Windows
  • Mac OSX
  • Linux

Description of the bug

Using the mouse wheel to scroll the Split Transactions dialog is very slow compared to scrolling in all other parts of MMEX and in all other applications I use.

Reproduction

Reproduction steps:

  1. Add sufficient rows on the Split Transactions dialog to require scrolling to view them all.
  2. Scroll up or down using the mouse wheel. Observe that the scrolling rate is unexpectedly and unreasonably slow.

For comparison, here are two screen recordings:

  1. Scrolling in the Split Transaction dialog; firstly using the scroll bar and then using the mouse wheel, spinning it as fast as possible.
  2. Scrolling in the Organize Categories dialog using the mouse wheel.
mmex_DLVuWqjrot.mp4
mmex_Hb3l7JhCco.mp4
@whalley
Copy link
Member

whalley commented Nov 27, 2022

I can't reproduce that lag under MacOS or Win11. E.g on Win11
CleanShot 2022-11-27 at 11 22 55

@tactilis
Copy link
Author

@whalley

I can't reproduce that lag under Win11

OK. Interesting.

What setting do you have for Lines to scroll at a time please?

image

As you can see, I have it set to 3, which is perfect for other apps and for other scrolling in MMEX - but not for Spilt Transactions. This video demonstrates the effect of turning the mouse wheel one 'notch' at a time. The scroll step on this particular dialog is tiny; it takes 12 wheel 'notches' to scroll one complete Split row.

mmex_JLeTDhzQ4y.mp4

I observe the same 12 steps to scroll one Split row if I use may laptop's trackpad instead of the mouse or if I use a different brand of mouse.

If I increase Lines to scroll at a time to, say 40, then close and restart MMEX, the Spilt Transactions dialog scrolls at a more reasonable rate (but is a bit jerky due to the larger steps) - but everywhere else scrolls too fast.

@whalley
Copy link
Member

whalley commented Nov 27, 2022

What setting do you have for Lines to scroll at a time please?

My settings are identical to yours.

The scroll step on this particular dialog is tiny; it takes 12 wheel 'notches' to scroll one complete Split row.

Yes, same for me if I scroll notch-by-notch.

whalley added a commit to whalley/moneymanagerex that referenced this issue Nov 27, 2022
@whalley
Copy link
Member

whalley commented Nov 27, 2022

See if above change helps.

@tactilis
Copy link
Author

SetScrollRate(6, 6);

Perfect. That makes scrolling comfortable.

Not that I ever have transactions with lots of splits - but we've seen that some people do, so this should help them.

@tactilis
Copy link
Author

The MMEX Options dialog has the same slow scroll rate. Scrolling is often needed in that dialog if View | Font size is set to Large or Huge.

Could the same SetScrollRate fix be applied please?

@whalley whalley added this to the v1.6.2 milestone Nov 27, 2022
whalley added a commit to whalley/moneymanagerex that referenced this issue Nov 27, 2022
@whalley whalley added the fixed label Nov 27, 2022
whalley added a commit that referenced this issue Nov 28, 2022
@whalley whalley self-assigned this Nov 28, 2022
@tactilis
Copy link
Author

tactilis commented Nov 28, 2022

Thanks. All good now:

  • Split Transactions
  • Options
  • Custom Fields

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants