Review Request 109210: Do not skip rows in Details View when scrolling by clicking the empty area of the scroll bar

Frank Reininghaus frank78ac at googlemail.com
Thu Feb 28 18:04:56 GMT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109210/
-----------------------------------------------------------

Review request for Dolphin.


Description
-------

Currently, the view is scrolled up or down by its height when the empty area of the scroll bar is clicked. However, this height includes the height of the header widget in Details View. Therefore, we skip a row when scrolling up or down.

I propose to not just use "view->size().height()" in KItemListController to determine the scroll bar's "pageStep", but to use a new member function of KItemListView which takes the possible header widget into account.


This addresses bug 315619.
    http://bugs.kde.org/show_bug.cgi?id=315619


Diffs
-----

  dolphin/src/kitemviews/kitemlistcontainer.cpp 5500851 
  dolphin/src/kitemviews/kitemlistview.h 2baf20c 
  dolphin/src/kitemviews/kitemlistview.cpp 6a68ae7 

Diff: http://git.reviewboard.kde.org/r/109210/diff/


Testing
-------

Works.


Thanks,

Frank Reininghaus

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130228/661efc24/attachment.htm>


More information about the kfm-devel mailing list