Project

General

Profile

Bug #6774

WTreeView::scrollTo() and more...

Added by Thomas Frank almost 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
11/28/2018
Due date:
% Done:

0%

Estimated time:

Description

The documentation of WTreeView says that 'EnsureVisible' provides minimal scrolling, but WTreeView::scrollTo() does exactly the opposite (scrolling 'toTop' if desired entry is below the bottom visible entry and vice versa). In addition there is some wrong calculation in getIndexRow() leading to wrong viewport adjustment in scrollTo().

Attached you will find a version of WTreeView.C (based on release 4.0.4) which fixes these issues. Watch out for comments starting with 'TF :'.

I'm still busy hunting down some other bugs in TreeView.C, which lead to wrong calculations of the '_viewport' and constant display of 'loading...' in the browser, especially when scrolling through large trees. This is with and without the attached fixes.


Files

WTreeView.C (78.1 KB) WTreeView.C Thomas Frank, 11/28/2018 04:32 PM
WTreeView.C (78.3 KB) WTreeView.C Thomas Frank, 11/29/2018 01:55 PM
WTreeView.C (78.9 KB) WTreeView.C Thomas Frank, 11/29/2018 03:06 PM
issue_6774.patch (5.39 KB) issue_6774.patch Roel Standaert, 02/03/2020 06:20 PM

Also available in: Atom PDF