dotnet / maui

.NET MAUI is the .NET Multi-platform App UI, a framework for building native device applications spanning mobile, tablet, and desktop.
https://dot.net/maui
MIT License
21.98k stars 1.72k forks source link

[Android] RefreshView disappears when switching tabs in a TabbedPage #17592

Open mfeingol opened 11 months ago

mfeingol commented 11 months ago

Description

When a RefreshView is hosted inside a TabbedPage, the RefreshView's progress circle disappears when switching between tabs.

Also, as a bonus bug: why is the RefreshView's progress circle half occluded by the tabs?

image

Steps to Reproduce

  1. Get a RefreshView spinning in a tab.
  2. Switch to the other tab. Go back.
  3. The RefreshView is still refreshing behind the scenes, but its progress circle has vanished

Link to public reproduction project repository

https://github.com/mfeingol/repros/tree/master/RefreshViewInTabbedPageRepro

Version with bug

8.0.0-rc.1.9171

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

Android

Affected platform versions

13

Did you find any workaround?

No.

Relevant log output

No response

ghost commented 11 months ago

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

XamlTest commented 11 months ago

Verified this on Visual Studio Enterprise 17.8.0 Preview 2.0(8.0.0-rc.1.9171). Repro on Android 13.0-API33 with below Project: RefreshViewInTabbedPageRepro.zip

samhouts commented 11 months ago

There are two bugs here. The one where the indicator is partially obscured is the same as #17610. However, the behavior where the indicator disappears when you click between tabs is a separate issue that is not a regression (reproduces in NET 7). Could be that the pages are disposing when the tabs are changed to save memory.

ghost commented 11 months ago

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

mfeingol commented 11 months ago

@samhouts: with respect, I'd suggest you expand your criteria for regressions to include areas that worked in Xamarin Forms and don't in .NET MAUI, regardless of whether they were already broken in previous versions of MAUI. Those of us who are porting Xamarin Forms apps to MAUI in advance of the oncoming loss of support won't really see a distinction - we just see broken functionality.

samhouts commented 11 months ago

Oh, worry not! If you tell me it was working in Xamarin.Forms, I'll add the migration-compatibility label. We're treating those with a similar level of priority (albeit slightly deferred over the next couple of weeks, since we're buttoning up .NET 8 GA right now).

Thanks!

mfeingol commented 11 months ago

@samhouts: it was working on Xamarin Forms. Thank you, and understand the tight schedule for GA.