[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Paolo Zamponi changed: What|Removed |Added CC||feu...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 iodream...@gmail.com changed: What|Removed |Added CC||iodream...@gmail.com --- Comment #19 from iodream...@gmail.com --- Does a bug report exist on QT's side? Can't seem to find anything. Thanks. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Gurenko Alex changed: What|Removed |Added CC||agure...@protonmail.com -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #18 from Nate Graham --- I'd prefer to do that only there's actually a bug report upstream, so we know they're aware it's something we'd like to change. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 fanzhuyi...@gmail.com changed: What|Removed |Added CC||fanzhuyi...@gmail.com --- Comment #17 from fanzhuyi...@gmail.com --- (In reply to Nate Graham from comment #15) > In progress with > https://invent.kde.org/plasma/plasma-integration/-/merge_requests/44! Based on the discussion in the MR, the consensus is to do this in Qt. So should we mark this as RESOLVED-UPSTREAM? -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 postix changed: What|Removed |Added CC||pos...@posteo.eu -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Status|ASSIGNED|CONFIRMED -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Assignee|uhh...@gmail.com|plasma-b...@kde.org -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #16 from Nate Graham --- > Btw, should I take the assignee position for the issue? I haven't seen any > more work being done on this by others Don't worry about it. :) -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Status|CONFIRMED |ASSIGNED --- Comment #15 from Nate Graham --- In progress with https://invent.kde.org/plasma/plasma-integration/-/merge_requests/44! -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #14 from Mauricius --- (In reply to Noah Davis from comment #10) > (In reply to Mauricius from comment #9) > > (In reply to Nate Graham from comment #8) > > > Interesting ideas! Feel free to submit a merge request with your idea to > > > https://invent.kde.org/plasma/breeze/-/merge_requests/. > > > > Thanks! I still have to tweak the system a bit since it has edge cases when > > you reach the end of page and simultaneously land on a scrollable control, > > but I have a more elegant solution in mind that would handle those cases > > too. > > > > The event filter is implemented in one header and one cpp file and can be > > installed by the app developer too, on any platform. Are you sure it would > > be best to implement it in Breeze? I'm new to KDE development so I wondered > > if there is at least some lower level KDE library that's used by all Qt > > apps, independent of system style. It would probably be more flexible to > > implement it in such place if possible. > > I suppose the Plasma Qt platform theme would be lower level than the QStyle. > The repo is https://invent.kde.org/plasma/plasma-integration/ Hi, I haven't updated anything regarding this bug in a long time, but I found some free time and after struggling with debugging the KDEPlasmaPlatformTheme library I confirmed that the method works. I don't know if this hack is the best way of solving this and whether there has been some work on the Qt side, but for now this simple patch seems to improve usability system-wide. I don't know what's the best way to test the code thoroughly except by replacing the library on runtime, which often crashes the DE (since I'm a n00b it seems), so some more testing might be required. I submitted a merge request https://invent.kde.org/plasma/plasma-integration/-/merge_requests/44 Btw, should I take the assignee position for the issue? I haven't seen any more work being done on this by others -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Jan Blackquill changed: What|Removed |Added Assignee|plasma-b...@kde.org |uhh...@gmail.com CC||uhh...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #13 from Nate Graham --- What we want here is that scrolling on controls still changes them, but if you scroll the view that a control is on, and that scroll action happens to move the cursor over a scrollable control, the control should ignore it and the view should keep on scrolling. Personally I find this to be a rather fiddly solution, and I think it would be simpler and more predictable to only allow scrolling on controls to change them when the controls are focused. But that would be a functional change for people used to the status quo, and a regression for controls *not* on scrollable views. So it's probably not something we can do. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 David Edmundson changed: What|Removed |Added CC||k...@davidedmundson.co.uk --- Comment #12 from David Edmundson --- Note for comboboxes it's easy, you can set QStyle::SH_ComboBox_AllowWheelScrolling to false. If we want anything else, we should be looking at patching Qt first not introducing hacks on our side. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added CC||natalie_clar...@yahoo.de --- Comment #11 from Nate Graham --- *** Bug 445239 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 magib...@hotmail.com changed: What|Removed |Added CC||magib...@hotmail.com -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #10 from Noah Davis --- (In reply to Mauricius from comment #9) > (In reply to Nate Graham from comment #8) > > Interesting ideas! Feel free to submit a merge request with your idea to > > https://invent.kde.org/plasma/breeze/-/merge_requests/. > > Thanks! I still have to tweak the system a bit since it has edge cases when > you reach the end of page and simultaneously land on a scrollable control, > but I have a more elegant solution in mind that would handle those cases too. > > The event filter is implemented in one header and one cpp file and can be > installed by the app developer too, on any platform. Are you sure it would > be best to implement it in Breeze? I'm new to KDE development so I wondered > if there is at least some lower level KDE library that's used by all Qt > apps, independent of system style. It would probably be more flexible to > implement it in such place if possible. I suppose the Plasma Qt platform theme would be lower level than the QStyle. The repo is https://invent.kde.org/plasma/plasma-integration/ -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #9 from Mauricius --- (In reply to Nate Graham from comment #8) > Interesting ideas! Feel free to submit a merge request with your idea to > https://invent.kde.org/plasma/breeze/-/merge_requests/. Thanks! I still have to tweak the system a bit since it has edge cases when you reach the end of page and simultaneously land on a scrollable control, but I have a more elegant solution in mind that would handle those cases too. The event filter is implemented in one header and one cpp file and can be installed by the app developer too, on any platform. Are you sure it would be best to implement it in Breeze? I'm new to KDE development so I wondered if there is at least some lower level KDE library that's used by all Qt apps, independent of system style. It would probably be more flexible to implement it in such place if possible. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #8 from Nate Graham --- Interesting ideas! Feel free to submit a merge request with your idea to https://invent.kde.org/plasma/breeze/-/merge_requests/. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Mauricius changed: What|Removed |Added CC||mauriciosmit...@gmail.com --- Comment #7 from Mauricius --- (In reply to Nate Graham from comment #6) > Looks like it'll be more complicated here, as I don't see any public API to > override the scroll behavior. We'll need to either override wheelEvent() for > these controls in our QStyle to conditionally do nothing, or else make a Qt > change to either do that or add API there that we can use downstream. Hi, new guy here! After a couple hours of toying with a dummy scrollception app, I think I found a way to circumvent the problem in the classic QWidgets apps that doesn't break the old functionality but prevents accidental inputs from happening. Instead of messing with Breeze, I implemented an app wide event filter that redirects the scroll events to the widget the user would expect to react. So this is what user sees: if the mouse is moved into a scrollable field, it should react by changing its value as normal. HOWEVER if the user keeps the mouse still and simply scrolls over the field, it will not react, but the widget that started scrolling would continue scrolling. This also applies to scroll-areas within scroll-areas: scrolling over one will not affect it, just like in browsers. How it's done: with an event filter installed on QApp. It remembers one widget as the scroll-active one and if the receiver of an event is not scroll-active, the event will be resent to the scroll-active one. This is how it responds to events: QMouseMoveEvent: * Reset scroll-active to 0 when the mouse moves * This ensures that the next scroll event will be propagated to the widget under cursor * On the next scroll event the receiver object should be set as the scroll-active widget QWheelEvent * If the scroll-active widget is not scrollable anymore (for example reached the end), reset scroll-active to 0 * for example, allow scrolling parent scroll-area if the current one reached its end * In case the receiver widget is a non-scrollable widget inside a scrollable area, find the first scrollable parent * Change the scroll-active widget to the receiver (or its parent) only if the scroll-active variable has been reset * i.e. prevent changing the scroll-active variable unless explicitly required * If the receiver isn't the scroll-active widget, redirect the event to the scroll-active one This of course isn't related to Breeze style but to Qt itself, but if we really want to apply this globally the style could perhaps force the event filter on the app it's used on. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #6 from Nate Graham --- Oops, that was for the wrong bug report. Regardless, I'm looking into implementhing nyanpasu64's suggestion. It turned out to be simple for our QtQuick controls; see https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/350 and https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/99. Looks like it'll be more complicated here, as I don't see any public API to override the scroll behavior. We'll need to either override wheelEvent() for these controls in our QStyle to conditionally do nothing, or else make a Qt change to either do that or add API there that we can use downstream. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 --- Comment #5 from Nate Graham --- Good idea! I think I have a way to fix this that will avoid upsetting the small number of people who would otherwise complain that this broke their workflow: we only allow scrolling on a control to change its value when it's focused. Then you can focus a control to fine-tune its value with a scroll, but unfocused controls will ignore scroll events to avoid the conflict between "view scroll" and "change control values scroll". -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Component|general |QStyle CC||noaha...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Karl Szmutny changed: What|Removed |Added CC||sha...@privy.de -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Patrick Silva changed: What|Removed |Added CC||squidin...@gmail.com --- Comment #4 from Patrick Silva --- *** Bug 418522 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Priority|NOR |HI --- Comment #3 from Nate Graham --- Proposal: disable this behavior for controls that are located inside a scrollview when the content is scrollable, since in this case scrolling to see more content can modify the controls unintentionally. At other times, allow a scroll to modify the controls' values. -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Severity|normal |major -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added CC||evald.spedz...@gmail.com --- Comment #2 from Nate Graham --- *** Bug 411611 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added CC||lu...@lwsabota.com --- Comment #1 from Nate Graham --- *** Bug 400350 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Patrick Silva changed: What|Removed |Added CC||bugsefor...@gmx.com -- You are receiving this mail because: You are watching all bug changes.
[Breeze] [Bug 399324] Scrolling changes sliders, list widgets, and numeric fields, causing unintentional changes to controls on scrollable views
https://bugs.kde.org/show_bug.cgi?id=399324 Nate Graham changed: What|Removed |Added Ever confirmed|0 |1 Component|general |general Product|kde |Breeze Summary|Mouse wheel changes |Scrolling changes sliders, |sliders, list widgets, and |list widgets, and numeric |numeric fields |fields, causing ||unintentional changes to ||controls on scrollable ||views Keywords||usability CC||n...@kde.org Status|REPORTED|CONFIRMED Assignee|unassigned-b...@kde.org |plasma-b...@kde.org -- You are receiving this mail because: You are watching all bug changes.
