Skip to content

998288 - Expand collapse of groups fails intermittently when listview is nested inside PullToRefresh on certain Android #318

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.

Sign up for GitHub

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

Merged
PaulAndersonS merged 4 commits into from
Jan 12, 2026
Merged

998288 - Expand collapse of groups fails intermittently when listview is nested inside PullToRefresh on certain Android #318

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3bef760
998288 - Expand collapse of groups fails intermittently when listview…
Monika4841 Jan 9, 2026
a389275
Added comment.
Monika4841 Jan 9, 2026
273f452
Update .NET SDK version from 9.0 to 10.0
PaulAndersonS Jan 9, 2026
b0ed5b3
Added the changes
Monika4841 Jan 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions maui/src/PullToRefresh/SfPullToRefresh.Android.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,8 @@ bool HandleActionMove(MotionEvent ev, Point currenTouchPoint)
_isChildScrolledVertically = IsChildElementScrolled(PullableContent.GetVisualTreeDescendants().FirstOrDefault(), new Point(ev.RawX / _density, ev.RawY / _density));
}

if (_downY < currenTouchPoint.Y && !_isChildScrolledVertically)
// 998288 - A slight downward movement during a tap is treated as a pull gesture, So add 10px vertical threshold before treating as pull gesture.
if ((currenTouchPoint.Y - this._downY) >= 10 && _downY < currenTouchPoint.Y && !_isChildScrolledVertically)
{
return true;
}
Expand Down Expand Up @@ -134,10 +135,10 @@ void ConfigTouch()
{
if (Handler is not null && Handler.PlatformView is not null)
{
// When the handler is changed without creating a new virtual view, adding a touch listener will skip attaching touch wiring to the native view.
// This is because the TouchDetector will still remain in the VirtualView and will be wired with the old handler and old native view.
this.RemoveTouchListener(this);
this.AddTouchListener(this);
// When the handler is changed without creating a new virtual view, adding a touch listener will skip attaching touch wiring to the native view.
// This is because the TouchDetector will still remain in the VirtualView and will be wired with the old handler and old native view.
this.RemoveTouchListener(this);
this.AddTouchListener(this);
_nativeView = Handler.PlatformView as LayoutViewGroupExt;

if (_nativeView is not null && _nativeView.Resources is not null && _nativeView.Resources.DisplayMetrics is not null)
Expand Down
Loading