Bug #6774
closedWTreeView::scrollTo() and more...
0%
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
Updated by Thomas Frank about 6 years ago
- File WTreeView.C WTreeView.C added
Slight correction to WTreeView::scrollTo() in respect to partially visible rows.
Updated by Thomas Frank about 6 years ago
- File WTreeView.C WTreeView.C added
Just found out, that the latest version of WTreeView.C at github fixes all the 'loading...'-issues! Great!!
I incorporated the fixed versions of WTreeView::scrollTo() and WTreeView::getIndexRow() into the latest WTreeView.C from github.
I know that this is now the 3rd attached version, but I hope that these fixes will make it into the next version anyway...
Updated by Roel Standaert about 6 years ago
- Target version changed from 4.0.5 to 4.1.0
Sorry for bumping this issue to the next version, but we'd already promised we would do a Wt 4.0.5 release this month and we were running out of time.
Updated by Koen Deforche about 5 years ago
- Status changed from New to InProgress
- Assignee set to Roel Standaert
Updated by Roel Standaert almost 5 years ago
- Target version changed from 4.1.0 to 4.2.1
Updated by Roel Standaert almost 5 years ago
- File issue_6774.patch issue_6774.patch added
I turned this into a patch, since that is a bit easier to digest.
Updated by Roel Standaert almost 5 years ago
- Target version changed from 4.2.1 to 4.3.0
Updated by Roel Standaert almost 5 years ago
- Status changed from InProgress to Resolved
It took me a little while to review these fixes. I can finally conclude that they indeed look correct. I pushed the fixes.
Updated by Roel Standaert almost 5 years ago
- Status changed from Resolved to Closed