https://bugs.kde.org/show_bug.cgi?id=386379
Nate Graham <n...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|CONFIRMED |RESOLVED
Resolution|--- |FIXED
Latest Commit| |https://commits.kde.org/dol
| |phin/403de19d9c036dd28481d3
| |b62bdb0f49f0792fbf
Version Fixed In| |19.12.1
--- Comment #49 from Nate Graham <n...@kde.org> ---
Git commit 403de19d9c036dd28481d3b62bdb0f49f0792fbf by Nate Graham.
Committed on 21/12/2019 at 18:54.
Pushed by ngraham into branch 'release/19.12'.
Improve scroll wheel speed by basing it on label height, not icon height
Summary:
Dolphin currently scrolls by the height of three items at a time per "step"
when
using a scroll wheel. Because item height is highly variable, this leads to
scroll
speed being inconsistent between views, and generally far too fast when using
icon view with icons larger than 22px size.
This patch makes the size of the scroll step based on the text label rather
than the
icon size just like D25683, ensuring that the scroll speed does not vary and
become
super fast when using large icons in particular.
It also reverts 90beb4a5e37b887caad1e767046a42dad0af1ab3, which is no longer
needed.
FIXED-IN: 19.12.1
Test Plan:
Use a mouse with a scroll wheel and scroll in Dolphin item views with list
view,
details view, icon view, etc, using different item sizes. Speed should be
consistent in all views now, and also feel consistent with other KDE apps.
Also try with multiple scale factors to make sure the behavior does not change.
No change with high-resolution two-finger touchpad scrolling.
Reviewers: #dolphin, elvisangelaccio
Reviewed By: #dolphin, elvisangelaccio
Subscribers: ahiemstra, lots0logs, anthonyfieroni, kfm-devel
Tags: #dolphin
Differential Revision: https://phabricator.kde.org/D19190
M +9 -1 src/kitemviews/kitemlistcontainer.cpp
M +0 -5 src/kitemviews/kitemlistview.cpp
M +0 -5 src/kitemviews/kitemlistview.h
M +0 -14 src/kitemviews/private/kitemlistsizehintresolver.cpp
https://commits.kde.org/dolphin/403de19d9c036dd28481d3b62bdb0f49f0792fbf
--
You are receiving this mail because:
You are watching all bug changes.
