New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Android] ScrollViewer shouldn't pass PointerMoved to parents #4244
[Android] ScrollViewer shouldn't pass PointerMoved to parents #4244
Conversation
The build 20437 found UI Test snapshots differences: Details
|
The build 20437 found UI Test snapshots differences: Details
|
The build 20437 found UI Test snapshots differences: Details
|
The build 20437 found UI Test snapshots differences: Details
|
The build 20437 found UI Test snapshots differences: Details
|
d8b7219
to
a533c5b
Compare
The build 20507 found UI Test snapshots differences: Details
|
The build 20507 found UI Test snapshots differences: Details
|
The build 20507 found UI Test snapshots differences: Details
|
The build 20507 found UI Test snapshots differences: Details
|
The build 20507 found UI Test snapshots differences: Details
|
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
The build 20636 found UI Test snapshots differences: Details
|
The build 20636 found UI Test snapshots differences: Details
|
The build 20636 found UI Test snapshots differences: Details
|
The build 20636 found UI Test snapshots differences: Details
|
The build 20636 found UI Test snapshots differences: Details
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's weird is that it's actually the exact event handled by the UnoTwoDScrollView ... not sure to understand how it comes back to the managed code 🤔. But indeed while scrolling pointer events are muted, so it's legit to handle it here.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
The build 20798 found UI Test snapshots differences: Details
|
The build 20798 found UI Test snapshots differences: Details
|
The build 20798 found UI Test snapshots differences: Details
|
The build 20798 found UI Test snapshots differences: Details
|
The build 20807 found UI Test snapshots differences: Details
|
The build 20807 found UI Test snapshots differences: Details
|
The build 20807 found UI Test snapshots differences: Details
|
The build 20807 found UI Test snapshots differences: Details
|
The build 20807 found UI Test snapshots differences: Details
|
46ea2d4
to
42bd0d1
Compare
42bd0d1
to
a4f11e4
Compare
The build 20874 found UI Test snapshots differences: Details
|
The build 20874 found UI Test snapshots differences: Details
|
The build 20874 found UI Test snapshots differences: Details
|
The build 20874 found UI Test snapshots differences: Details
|
The build 20874 found UI Test snapshots differences: Details
|
The build 20922 found UI Test snapshots differences: Details
|
The build 20922 found UI Test snapshots differences: Details
|
GitHub Issue (If applicable): #
PR Type
What kind of change does this PR introduce?
What is the current behavior?
What is the new behavior?
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
Internal Issue (If applicable): fixes https://github.com/unoplatform/nventive-private/issues/3