https://bugs.kde.org/show_bug.cgi?id=386379

--- Comment #55 from Nate Graham <n...@kde.org> ---
I understand the frustration here. However please understand how frustrated
people were with the previous behavior! :) Before the fix for this bug report,
Dolphin did take into account the icon size when scrolling with a wheel and
enough people hated it that we got four bug reports.

So we have a few options here:
1. Revert the fix and make everyone unhappy who thought the scroll speed with
large icons was too fast
2. Bring back the old code to speed up scrolling based on icon size, but tweak
the algorithm to make the scrolling speed a bit slower than it used to be, or
scale it up less quickly than the icon size increases, or something else
3. Make it configurable
4. Give up and say that it's the responsibility of the toolkit or input driver


Let's go through the options:

#1 makes people unhappy enough to file lots of bug reports. Clearly a no-go.

#2 is possible, but suffers from a common problem when you try to make two
groups of people happy who want opposite things: often you wind up pleasing
neither group. However maybe it is possible with enough tweaking.

#3 is possible, but we generally don't like to add options like this to apps
unless it's absolutely necessary, because most people never touch the settings
and will be frustrated anyway, and every user-facing setting you add to a
config window makes the window a little big more visually overwhelming and
people already complain that KDE software's configurability is excessive and
intimidating.

#4 is arguably the correct fix since Dolphin is not the only app to suffer from
this issue. For example scrolling long documents in any text editor or word
processor app is agonizing for the same reason, because each tick of the scroll
wheel goes exactly three lines. However making it the toolkit or input driver's
responsibility means no fix for the foreseeable future until developers figure
out how to implement the real fix at a lower level.

As you can see, there is no obviously perfect option. Options that stand out to
me are #2 and #4. I personally think that the correct solution is to make the
wheel scroll speed configurable globally, not just here in Dolphin. However I'm
open to #2 if anyone has any ideas--preferably in patch form. :) The code
change to do this was not particularly complicated and the former code that
scaled the scroll speed with large icons was reasonably comprehensible.

The previous code did:

singleStep = view->itemSizeHint().height();

Maybe we could try with:

const QFontMetrics metrics(font());
singleStep = metrics.height() * QApplication::wheelScrollLines();
singleStep = max(view->itemSizeHint().height() / 2, metrics.height() *
QApplication::wheelScrollLines());

This would keep the previous scaling behavior, but make the scaling only half
as big as it was before, and keep the minimum number of lines to scroll
feature.

Feel free to submit patches

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to