[Android & iOS]: In ListView with IsPullToRefreshEnabled enabled the PullToRefresh activity indicator remains stuck and doesnt get closed in .net 8 MAUI about maui HOT 12 CLOSED

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Open similar issues:

• PullToRefresh activity indicator hidden behind header in CollectionView (#20723), similarity score: 0.79
• ListView pull-to-refresh not working in Windows (WinUI) but work on Android/iOS (#18850), similarity score: 0.77
• [MAUI] RefreshView C6 or C7 Pull down in the view isn't activate the RefreshView on Surface. (#16974), similarity score: 0.76

Closed similar issues:

• [regression/8.0.0-preview.7.8842] - RefreshView does not hide spinner when IsRefreshing set to false. (#16910), similarity score: 0.78
• RefreshView on Android at .net 8 preview 7 don't stop refreshing (#17338), similarity score: 0.77

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

from maui.

@muffadal53 actually it's the other way around, I was mistaken. The ListView sets the IsRefreshing for you. So what happens is that the ListView sets IsRefreshing to true before it gets to this check: https://github.com/muffadal53/ListViewRefreshApp/blob/master/ViewModel/TestConfigViewModel.cs#L33

That check will evaluate to being false and then entire thing is being skipped and IsRefreshed is never set to false.

This is not a bug in .NET MAUI but rather something that is not correct in your code.

Thanks for the report!

from maui.

@samhouts: This issue is not resolved.So, can you please reopen this issue.Thanks.

from maui.

It is really surprising that without providing working solution or workaround MAUI issues are just getting closed. Please provide answer to this issue @jfversluis, @davidortinau, @samhouts.

from maui.

Verified this issue with Visual Studio 17.10 Preview 7 (8.0.40 & 8.0.3). Can repro it.

from maui.

@jfversluis: Can you please help in fixing this issue with workaround or urgent service pack release of .NET 8.0 as it is common usecase to use PullToRefresh functionality of ListView and it is urgent for our app release?

from maui.

same error here

from maui.

Is there any reason you're setting Mode=OneWay on the IsRefreshing property? I think that might be the issue here. The ListView will set that to false whenever it's done refreshing, but now it can't because you set the mode to OneWay? Have you tried that?

from maui.

Is there any reason you're setting Mode=OneWay on the IsRefreshing property? I think that might be the issue here. The ListView will set that to false whenever it's done refreshing, but now it can't because you set the mode to OneWay? Have you tried that?

@jfversluis: The reason I have set the Mode=OneWay is because I am managing the value of IsRefreshing from the ViewModel during Reload operation. And even if you remove the Mode=OneWay the IsRefreshing spinner remains stuck
And if you look at the documentation of MAUI it is contradicting to what you have mentioned in your comment and here is what it is says in the documentation "Once the ListView is refreshed, the IsRefreshing property should be set to false, or the EndRefresh method should be called on the ListView, to indicate that the refresh is complete."

So, please let me know how this issue can be Awaiting prompt reply.

from maui.

@jfversluis: Yes it is other way round. The "IsRefreshing" is set to true by the ListView and we need to set it to "false" after Refresh. And I have updated my code and below is the updated version. Now, the "IsRefreshed" is set to false but even with that change the "IsRefreshing" spinner doesn't get closed.

Below us the updated code in the ViewModel:

`private void RefreshListItems()
{
try
{
TestConfigList.Clear();
ReloadFromSource();
}
finally
{
ListRefreshing = false;
}

}`

You can check the repositry too:
https://github.com/muffadal53/ListViewRefreshApp/blob/master/ViewModel/TestConfigViewModel.cs#L33

So, without checking the code by making the change you are suggesting as not an issue you just closed the issue by giving the reference of the MAUI document. This doesn't seem to be right. And it is not only me who has got this issue. Along with me @ninachen03 and @eynerdevp also are able to reproduce the issue. So, please use my updated version of the ListView sample and run it on your side and let me know the behavior.

@davidortinau: Please look into this issue and let me know the solution as it is urgent on our side.

Awaiting feedback at the earliest. And this issue is not Closed till the solution is provided.

Regards

from maui.

I pulled down your code again and added OnPropertyChanged(nameof(ListRefreshing)); right after you set ListRefreshing = false;. This makes it work as it should.

There is no issue with .NET MAUI, this works as intended, therefore it's closed. The issue is somewhere in the logic in your code unfortunately.

from maui.

Thanks @jfversluis for the clarification. I think it would be better to have a concrete example in MAUI documentation of ListView showing in section of https://learn.microsoft.com/en-us/dotnet/maui/user-interface/controls/listview?view=net-maui-8.0#pull-to-refresh Pull to refresh on how this property must be handled. The current example doesn't seem to explain this detail properly.

from maui.