Project

General

Profile

Actions

Improvements #13163

closed
AA RM

WTableView touch selection behavior

Improvements #13163: WTableView touch selection behavior

Added by Andrey Alekseev almost 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
10/29/2024
Due date:
% Done:

100%

Estimated time:

Description

When scrolling WTableView contents on mobile, it changes selection before touch is released.
According to WTableView.js, current implementation only waits 0.5-1 sec between start end end of a touch. Of course it works when user uses kinetic scroll - gives some acceleration to a scroll area with one fast swipe. But when trying to scroll slowly, selection changes when it shouldn't.
I wonder why Signal<WModelIndex, WMouseEvent>& clicked() is not used for selection instead. I connected a simple logger to it, and I didn't see any false activations while scrolling.

MV Updated by Matthias Van Ceulebroeck almost 2 years ago Actions #1

  • Target version set to 4.11.3

Hey Andrey,

thanks for the report. We'll take a look at it soon!

RM Updated by Romain Mardulyn over 1 year ago Actions #2

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

RM Updated by Romain Mardulyn over 1 year ago Actions #3

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

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions #4

  • Assignee set to Matthias Van Ceulebroeck

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions #5

  • Target version changed from 4.11.3 to 4.11.4

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions #6

  • Target version changed from 4.11.4 to 4.12.0

MV Updated by Matthias Van Ceulebroeck about 1 year ago Actions #7

  • Status changed from Review to Implemented @Emweb
  • Assignee changed from Matthias Van Ceulebroeck to Romain Mardulyn
  • % Done changed from 0 to 100

MV Updated by Matthias Van Ceulebroeck about 1 year ago Actions #8

  • Status changed from Implemented @Emweb to Implemented @Test

MV Updated by Matthias Van Ceulebroeck about 1 year ago Actions #9

  • Status changed from Implemented @Test to Closed
Actions

Also available in: PDF Atom