MethodicConfigurator icon indicating copy to clipboard operation
MethodicConfigurator copied to clipboard

Scroll with mouse wheel does not work over labels and text fields in components and in parameters

Open flyasky opened this issue 9 months ago • 15 comments

Is there an existing issue for this?

  • [x] I have searched the existing issues

Description of the bug

On Windows scroll is possible only on empty places without any child component.

Steps To Reproduce

No steps - it's just existing UI bug

Expected behavior

Would be handy to enable scrolling over all components except comboboxes.

Software information

  • Operating system: MS Windows
  • ArduPilot Methodic Configuration version: 1.4.4 and 2.6.1

flyasky avatar May 02 '25 18:05 flyasky

I had to disable mouse scrolling because it was wreaking havoc on comboboxes. It would be great to re-enable it, but keep it disabled over comboboxes like you said.

amilcarlucas avatar May 03 '25 14:05 amilcarlucas

Is this issue still on?

Siddharthh39 avatar May 26 '25 04:05 Siddharthh39

Yes, this has not been fixed yet. It it very low priority. @Siddharthh39 do you want to help me fix it?

amilcarlucas avatar May 26 '25 08:05 amilcarlucas

Yes, this has not been fixed yet. It it very low priority. @Siddharthh39 do you want to help me fix it?

Sure, It would a wonderful opportunity for me to learn.

Siddharthh39 avatar May 26 '25 08:05 Siddharthh39

See ardupilot_methodic_configurator/frontend_tkinter_component_editor.py:192-219

amilcarlucas avatar Jul 03 '25 19:07 amilcarlucas

@amilcarlucas if this issue is still open Would it be okay if I take this up?

Trishanshv avatar Aug 29 '25 14:08 Trishanshv

@Trishanshv sure have a go at it. The relevant code is here: https://github.com/ArduPilot/MethodicConfigurator/blob/fd360b67b1744c527babe7e158a0ff3ba714bc05/ardupilot_methodic_configurator/frontend_tkinter_component_editor.py#L182

amilcarlucas avatar Sep 01 '25 07:09 amilcarlucas

@amilcarlucas, This is my first contribution so I wanted to share my approach before implementing it.

I’m thinking of creating a small helper function that forwards mouse wheel events from entries and comboboxes back to their parent frame . please correct me if I am wrong .

Trishanshv avatar Sep 02 '25 12:09 Trishanshv

I think the best approach is to only accept the mouse wheel if the widget is selected, and if the widget is not selected, forward the events to their parent frame.

amilcarlucas avatar Sep 02 '25 12:09 amilcarlucas

@Trishanshv in 2.6.1 fix does not work on the parameter editor on Windows

flyasky avatar Oct 19 '25 18:10 flyasky

I tested it on both the component editor and the parameter editor on windows using the upcoming master branch. It was correctly you can use the mouse wheel to scroll everywhere.

Please post a video showing exactly what is not working correctly for you.

amilcarlucas avatar Oct 20 '25 22:10 amilcarlucas

Version 2.6.1, screen capture attached. https://github.com/user-attachments/assets/1af2a518-ace9-4563-800d-07a36d3f31c1

flyasky avatar Oct 21 '25 07:10 flyasky

I see a pattern here. On your computer ist stops scrolling whenever it opens a tooltip popup (yellow help text).

I do not remember seeing this on my computer, and I do have a slow windows computer. I will test it again.

amilcarlucas avatar Oct 21 '25 09:10 amilcarlucas

I tested it, but both my machines have Windows 11 and I can not reproduce it.

What is your exact windows version?

amilcarlucas avatar Oct 21 '25 09:10 amilcarlucas

I could reproduce it, but only on windows and only when using the windows installer to install the application.

This seams to be a pyinstaller issue/ race condition.

amilcarlucas avatar Oct 27 '25 08:10 amilcarlucas