Skip to content

Conversation

eminfedar
Copy link

  • If the change modifies a visible behavior, it changes the documentation accordingly
  • If possible, the change is auto-tested
  • If the changes fixes or close an existing issue, the commit message reference the issue with Fixes #xxx or Closes #xxx
  • If the change is noteworthy, the commit message should contain ChangeLog: ...

Fixes: #9521

@CLAassistant
Copy link

CLAassistant commented Sep 25, 2025

CLA assistant check
All committers have signed the CLA.

@ogoffart
Copy link
Member

ogoffart commented Sep 26, 2025

Thanks for submitting a PR.
The delay is there on purpose because while scrolling, we don't want the press event to activate.
Maybe the 100ms is too long and can be adjusted. Or we'd use other heuristic.
But i think the behaviour that you see in bug #9521 is precisely because we got complaints that element would be shown as pressed while scrolling
(it was added in #1652 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flickable (and ScrollView): Add a property to prevent delaying press events

3 participants