Preferences: disable scrolling when hovering comboboxes

Bug #1885628 reported by ronso0
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
ronso0

Bug Description

When scrolling through preferences pages it's inevitable that the cursor hovers a combobox. Unfortunately this can cause the combobox setting to change without the user noticing, like choosing an audio device.

Let's disable the scroll capabilty of comboboxes in the preferences ui files. Maybe <FocusPolicy>Qt::StrongFocus</FocusPolicy> is sufficient already.

related:
lp:1881493 "don't change effect when scrolling over WEffectSelector"

Revision history for this message
ronso0 (ronso0) wrote :

Any objections to ignore the scroll event on comboboxes?
If there are none I'll try to fix this.

Changed in mixxx:
assignee: nobody → ronso0 (ronso0)
milestone: none → 2.3.0
importance: Undecided → Low
Revision history for this message
Be (be.ing) wrote :

I would only want this for the Sound Hardware page. I stumble over this fairly frequently. On other pages I'd like my scroll wheel to work as normal.

Be (be.ing)
Changed in mixxx:
status: New → Confirmed
milestone: 2.3.0 → none
Revision history for this message
ronso0 (ronso0) wrote :

yeah, we need this only where
1) the pref page is long = likely to be scrolled to show hidden options
and
2) comboboxes span almost the entire width of the pref page
3) comboboxes control high-impact options (sound settings)

looks like this is true only for the sound hardware page.

Revision history for this message
Be (be.ing) wrote :

So this is just a quick hack. The real problem is that the Sound Hardware page is way too long.

Revision history for this message
ronso0 (ronso0) wrote :

Exactly. No scrolling, no problem :)

We could also split all sound settings into tabs:
General | Output | Input

Revision history for this message
ronso0 (ronso0) wrote :

Hints & Diagnositics could go to the General tab, may then both Output and Input don't need to be scrolled anymore.

Revision history for this message
Be (be.ing) wrote :

For 2.3 let's stick with the quick hack of disabling scrolling comboboxes in DlgPrefSound. If you want to redesign it, that's a bigger project than simply moving Hints & Diagnostics to a new tab.

Be (be.ing)
Changed in mixxx:
status: Confirmed → Fix Committed
milestone: none → 2.3.0
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/10033

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.