Skip to content

[regression/8.0.0-preview.7.8842] - RefreshView does not hide spinner when IsRefreshing set to false.Β #16910

Closed

Description

@RobTF

Description

The RefreshView control does not hide the spinner after the refresh operation completes and IsRefreshing is set to false. Same code worked fine in .NET 7.

Steps to Reproduce

  1. Clone this repo https://github.com/RobTF/mauiplay/tree/net8/CollectionViewIssue
  2. Run on Android
  3. Tap "Go to test page"
  4. Pull to refresh
  5. Observe refresh spinner does not disappear.

Link to public reproduction project repository

https://github.com/RobTF/mauiplay/tree/net8/CollectionViewIssue

Version with bug

8.0.0-preview.7.8842

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

7.0.92

Affected platforms

Android

Affected platform versions

Android 13, Android 14

Did you find any workaround?

No

Relevant log output

No response

Activity

  1. RobTF commented on Aug 22, 2023

    @RobTF
    Author

    Currently has us blocked trying .NET 8 any further.

  2. ghost added
    legacy-area-controlsLabel, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor
    on Aug 22, 2023
  3. changed the title [-]8.0.0-preview.7.8842 - RefreshView does not hide spinner when IsRefreshing set to false. [Regression from 7.0.92][/-] [+][regression/8.0.0] - RefreshView does not hide spinner when IsRefreshing set to false.[/+] on Aug 22, 2023
  4. added this to the .NET 8 GA milestone on Aug 22, 2023
  5. samhouts commented on Aug 22, 2023

    @samhouts
    Contributor

    Something like this has been happening for a while, it seems. See #8926

  6. DM-98 commented on Aug 22, 2023

    @DM-98

    @samhouts As it seems, it bugged out in .NET 6, then it has worked in:

    • .NET 7 (someone mentioned it worked)
    • .NET 8 Preview 5 (I personally used it and it worked)

    And then it stopped at .NET 8 Preview 6 and 7.

    I don't know why it's P/1, since Pull to refresh is a popular element used in apps.

  7. samhouts commented on Aug 28, 2023

    @samhouts
    Contributor

    p/1 is our highest priority (aside from p/0, which should be reserved for critical, "everything-is-on-fire" issues).

  8. changed the title [-][regression/8.0.0] - RefreshView does not hide spinner when IsRefreshing set to false.[/-] [+][regression/8.0.0-preview.7.8842] - RefreshView does not hide spinner when IsRefreshing set to false.[/+] on Sep 18, 2023
  9. ghost locked as resolved and limited conversation to collaborators on Oct 18, 2023
  10. removed
    legacy-area-controlsLabel, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor
    on May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions