D10102: base scrolling on the smallest item

Marco Martin noreply at phabricator.kde.org
Thu Jan 25 18:09:02 GMT 2018


mart created this revision.
mart added reviewers: Plasma, Dolphin.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
mart requested review of this revision.

REVISION SUMMARY
  after recent highdpi patches on scrolling that delegated it
  completely to the scrollbar, based upon the scrollbar singleStep
  setted to the tallest of the items in the view.
  tough this makes scrolling way too fast, and on folders where just
  few filenames are longer than most we can get a single scrolling
  step almost double the number of lines configured in the
  mouse kcm.
  Using the shortest item instead of the tallest mitigates this problem
  making it a bit more usable

TEST PLAN
  tested on different folders in different view modes both with
  mouse and touchpad

REPOSITORY
  R318 Dolphin

BRANCH
  phab/dolphinScroll

REVISION DETAIL
  https://phabricator.kde.org/D10102

AFFECTED FILES
  src/kitemviews/kitemlistcontainer.cpp
  src/kitemviews/kitemlistview.cpp
  src/kitemviews/private/kitemlistsizehintresolver.cpp
  src/kitemviews/private/kitemlistsizehintresolver.h

To: mart, #plasma, #dolphin
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180125/3b8ce168/attachment.htm>


More information about the kfm-devel mailing list