[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 Nate Graham changed: What|Removed |Added Latest Commit||https://invent.kde.org/plas ||ma/plasma-workspace/commit/ ||ff4fa345c5119000809c8bbb135 ||7285cfd168a1e Version Fixed In||5.22 Resolution|--- |FIXED Status|REPORTED|RESOLVED --- Comment #20 from Nate Graham --- Git commit ff4fa345c5119000809c8bbb1357285cfd168a1e by Nate Graham, on behalf of Bernhard Sulzer. Committed on 30/04/2021 at 16:14. Pushed by ngraham into branch 'master'. Fix battery applet high-res scrolling Rounding errors made scrolling behave weirdly for devices supplying very low scroll delta values. This is fixed now. For non-high resolution devices, scrolling is now also aligned to 5%, just like the keyboard shortcuts. FIXED-IN: 5.22 M +1-1applets/batterymonitor/package/contents/ui/BrightnessItem.qml M +17 -7 applets/batterymonitor/package/contents/ui/CompactRepresentation.qml https://invent.kde.org/plasma/plasma-workspace/commit/ff4fa345c5119000809c8bbb1357285cfd168a1e -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 soredake changed: What|Removed |Added CC||ndrzj1...@relay.firefox.com -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #19 from Bernhard --- (In reply to Nate Graham from comment #18) > Thanks for testing. Could you please post that as a comment on that merge > request? I know it's closed, but perhaps it can be re-opened. Also reference > this bug report by its URL. I have tried to really fix this properly and have started a merge request for that here: https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/632 Please let me know it there is anything wrong with it as I am not too experienced -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #18 from Nate Graham --- Thanks for testing. Could you please post that as a comment on that merge request? I know it's closed, but perhaps it can be re-opened. Also reference this bug report by its URL. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #17 from Bernhard --- Okay I fixed it. Basically the only thing that's missing is explicit (and actually correct) rounding in the last line of the function: Instead of this batterymonitor.screenBrightness = Math.max(minimumBrightness, Math.min(maximumBrightness, batterymonitor.screenBrightness + deltaSteps * steps)); I should be this batterymonitor.screenBrightness = Math.max(minimumBrightness, Math.min(maximumBrightness, batterymonitor.screenBrightness + Math.round(deltaSteps * steps))); What the patch is trying to do (storing the rounding error for future events) would be the correct thing to do, but the way that it's done makes it worse than just this simple fix. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #16 from Bernhard --- (In reply to Nate Graham from comment #14) > In this case the simplest thing would probably be to directly edit the file > on disk--after making a backup of course! > > The file in question is > /usr/share/plasma/plasmoids/org.kde.plasma.battery/contents/ui/ > batterymonitor.qml. Open it up and edit it to mimic the changes in the diff > at > https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/590/diffs. > Lines in green are added, and lines in red are removed. > > After you do this, restart plasmashell with `plasmashell --replace` and see > if the changes helped. Okay I tested this and it works, kind of. It does however remove fine-grained control completely and also has rounding errors where (depending on current brightness) it changes 6% instead of the otherwise 5% increments. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #15 from Nate Graham --- (and if you're complaining about smooth scrolling in Okular, you can turn it off in the config window) -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #14 from Nate Graham --- In this case the simplest thing would probably be to directly edit the file on disk--after making a backup of course! The file in question is /usr/share/plasma/plasmoids/org.kde.plasma.battery/contents/ui/batterymonitor.qml. Open it up and edit it to mimic the changes in the diff at https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/590/diffs. Lines in green are added, and lines in red are removed. After you do this, restart plasmashell with `plasmashell --replace` and see if the changes helped. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #13 from Bernhard --- (In reply to Bernhard from comment #12) > It goes away when not using smooth scrolling, but I have to say that is quit > unpleasant because you lose the line you're reading. > > Smooth scrolling otoh is something I can only describe as extremely > unresponsive. It does not seem to accelerate or slow down to with the mouse > wheel but on it's own time. If I want to scroll one page exactly it will > slowly begin scrolling (no matter how fast the wheel goes) and then > decelerate (while the scroll wheel is already stopped) going pages further > than it should. Sorry that comment was meant for another bug report. Please ignore it (or delete, if possible) -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #12 from Bernhard --- It goes away when not using smooth scrolling, but I have to say that is quit unpleasant because you lose the line you're reading. Smooth scrolling otoh is something I can only describe as extremely unresponsive. It does not seem to accelerate or slow down to with the mouse wheel but on it's own time. If I want to scroll one page exactly it will slowly begin scrolling (no matter how fast the wheel goes) and then decelerate (while the scroll wheel is already stopped) going pages further than it should. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #11 from Bernhard --- > This might have been fixed by > https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/590 Any > chance you can test that? I would I if I knew how to. Is there a 'simple' way to compile it to a .deb file for neon? The build instructions (https://community.kde.org/Guidelines_and_HOWTOs/Build_from_source/Plasma_5_on_Ubuntu_14.04_LTS) seem pretty old and complicated. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #10 from Nate Graham --- I mean https://invent.kde.org/plasma/plasma-pa/-/blob/master/applet/contents/ui/main.qml#L306 and https://invent.kde.org/plasma/plasma-workspace/-/blob/master/applets/batterymonitor/package/contents/ui/batterymonitor.qml#L180 -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #9 from Nate Graham --- Yep, the implementations are different: https://invent.kde.org/plasma/plasma-workspace/-/blob/master/applets/batterymonitor/package/contents/ui/batterymonitor.qml#L180 https://invent.kde.org/plasma/plasma-workspace/-/blob/master/applets/batterymonitor/package/contents/ui/batterymonitor.qml#L180 This might have been fixed by https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/590 Any chance you can test that? -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #8 from Bernhard --- (In reply to Bernhard from comment #6) > I have not noticed similar behavior in any other application or applet. The > volume applet works perfectly and at a reasonable speed (for me) Okay, my answer was perhaps a bit hasty. 'Perfectly' is not exactly true, if I scroll to fast the volume applet seems to miss almost all scroll events. At any decently slow speed it works fine though, without weird direction dependent behavior. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 Nate Graham changed: What|Removed |Added Resolution|WAITINGFORINFO |--- Status|NEEDSINFO |REPORTED --- Comment #7 from Nate Graham --- All right, in that case there must be something odd with the scroll handling implementation specific to the battery monitor. Thanks for your patience. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #6 from Bernhard --- I have not noticed similar behavior in any other application or applet. The volume applet works perfectly and at a reasonable speed (for me) -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #5 from Nate Graham --- Thanks. I wonder if it's touchpad-specific, or a quirk of Libinput's scroll hysteresis. Does this behavior manifest in any other scrollviews? Or only when scrolling over the brightness applet? And what about when scrolling over the volume applet? -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #4 from Bernhard --- Considering this was a wayland session I would guess I used libinput - wouldn't know how to check tough. I've got the exact same problem on this system running Arch with 5.20.5 on X11 though. This is what xinput reports on Arch, I hope this helps: $ xinput list-props 'SYNA32A1:00 06CB:CE17 Touchpad' Device 'SYNA32A1:00 06CB:CE17 Touchpad': Device Enabled (158): 1 Coordinate Transformation Matrix (160): 0.00, 0.00, 0.00, 0.00, 0.00, 0.00, 0.00, 0.00, 0.00 Device Accel Profile (292): 1 Device Accel Constant Deceleration (293): 2.50 Device Accel Adaptive Deceleration (294): 1.00 Device Accel Velocity Scaling (295):12.50 Synaptics Edges (319): 53, 1279, 44, 784 Synaptics Finger (320): 25, 30, 0 Synaptics Tap Time (321): 180 Synaptics Tap Move (322): 58 Synaptics Tap Durations (323): 180, 180, 100 Synaptics ClickPad (324): 1 Synaptics Middle Button Timeout (325): 0 Synaptics Two-Finger Pressure (326):282 Synaptics Two-Finger Width (327): 7 Synaptics Scrolling Distance (328): 25, 25 Synaptics Edge Scrolling (329): 0, 0, 1 Synaptics Two-Finger Scrolling (330): 1, 1 Synaptics Move Speed (331): 1.977633, 255.00, 0.062808, 0.00 Synaptics Off (332):2 Synaptics Locked Drags (333): 0 Synaptics Locked Drags Timeout (334): 1000 Synaptics Tap Action (335): 0, 3, 0, 0, 1, 3, 2 Synaptics Click Action (336): 1, 3, 2 Synaptics Circular Scrolling (337): 0 Synaptics Circular Scrolling Distance (338):0.104720 Synaptics Circular Scrolling Trigger (339): 0 Synaptics Circular Pad (340): 0 Synaptics Palm Detection (341): 0 Synaptics Palm Dimensions (342):10, 200 Synaptics Coasting Speed (343): 20.00, 50.00 Synaptics Pressure Motion (344):30, 160 Synaptics Pressure Motion Factor (345): 1.00, 1.00 Synaptics Grab Event Device (346): 0 Synaptics Gestures (347): 1 Synaptics Capabilities (348): 1, 0, 0, 1, 1, 0, 0 Synaptics Pad Resolution (349): 12, 12 Synaptics Area (350): 0, 0, 0, 0 Synaptics Soft Button Areas (351): 666, 0, 678, 0, 0, 0, 0, 0 Synaptics Noise Cancellation (352): 7, 7 Device Product ID (285):1739, 52759 Device Node (284): "/dev/input/event6" If you want to know anything more specific you'd have to provide me with a command for that. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 Nate Graham changed: What|Removed |Added CC||n...@kde.org Resolution|--- |WAITINGFORINFO Status|REPORTED|NEEDSINFO --- Comment #3 from Nate Graham --- Works for me with my touchpad. What do you mean exactly by "high resolution scrolling?" What device are you using to scroll? And what drivers are you using? Libinput or Synaptics/Evdev? -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #2 from Bernhard --- Sorry, misinformation. Horizontal scrolling is in every way the same as vertical when it comes to buggy behavior. -- You are receiving this mail because: You are watching all bug changes.
[plasmashell] [Bug 432331] Weird behavior with high-resolution scrolling
https://bugs.kde.org/show_bug.cgi?id=432331 --- Comment #1 from Bernhard --- I only mentionend in the title: This is specific to high-resolution scrolling. Also: For some reason horizontal scrolling is also accepted for changing brightness. This has none of the problems of normal vertical scrolling while hovering over the panel. Inside the applet horizontal scrolling is exactly as bugged as vertical scrolling. After this is fixed I would also propose removing horizontal scrolling, as not scrolling perfectly straight down on a touchpad means you sometimes have unwanted 'side effects' when it is enabled. -- You are receiving this mail because: You are watching all bug changes.
