Project

General

Profile

Actions

Feature #13785

closed
MS RM

WTableView::scrollTo: Take column into account

Feature #13785: WTableView::scrollTo: Take column into account

Added by Michael Seibt over 1 year ago. Updated 6 months ago.

Status:
Closed
Priority:
High
Target version:
Start date:
06/19/2025
Due date:
% Done:

0%

Estimated time:

Description

As the documentation of Wt 4.11.4 admits (https://www.webtoolkit.eu/wt/doc/reference/html/classWt_1_1WTableView.html#a093b661f93551c1d6f73c423ae8ef459),
the current implementation of WTableView::scrollTo(WModelIndex& index) does not take index.column() into account.
We are porting a desktop application having wide tables. Single cell selection must work using arrow keys.
We have implemented this by handling signals keyWentDown and selectionChanged calling scrollTo.
As documented, WTableView::scrollTo(WModelIndex&) does not scroll horizontally.
WTableView::scrollTo(int x, int y) with x = /*magic number from WTableView.C*/ 7 + sum of column widths to the left and y = -1 is a workaround.
But it scrolls for every movement which results in a terrible UX.
The necessary information "which cells are visible in the browser" is available in WTableView, but inaccessible because all private.
I.e. WTableView itself needs to be modified.

MS Updated by Michael Seibt over 1 year ago Actions #1

Please do not duplicate the flaws of vertical scrolling as reported in #13786.

MV Updated by Matthias Van Ceulebroeck 11 months ago Actions #2

  • Target version set to 4.13.0

RM Updated by Romain Mardulyn 11 months ago Actions #3

  • Status changed from New to InProgress
  • Assignee set to Romain Mardulyn

RM Updated by Romain Mardulyn 10 months ago Actions #4

  • Status changed from InProgress to Review
  • Assignee deleted (Romain Mardulyn)

MS Updated by Michael Seibt 7 months ago Actions #5

RM Updated by Romain Mardulyn 6 months ago Actions #6

  • Status changed from Review to Implemented @Emweb
  • Assignee set to Romain Mardulyn

RM Updated by Romain Mardulyn 6 months ago Actions #7

  • Status changed from Implemented @Emweb to Closed
Actions

Also available in: PDF Atom