Skip to content
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

[MAUI] RefreshView C6 or C7 Pull down in the view isn't activate the RefreshView on Surface. #16974

Open
v-yanfw opened this issue Aug 24, 2023 · 6 comments
Labels
area-controls-refreshview RefreshView csi-new platform/windows 🪟 s/needs-attention Issue has more information and needs another look s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working testing-manual
Milestone

Comments

@v-yanfw
Copy link
Collaborator

v-yanfw commented Aug 24, 2023

Description

When refreshing RefreshView C6 or C7 page, RefreshView doesn't work on Surface.
Note: This issue does not repro on Android Simulator.

Steps to Reproduce

1.Deploy https://devdiv.visualstudio.com/Pkgs/_git/VS.TestAssets.Xaml?path=/Assets/Projects/ManualMauiTests to Windows.
2.Click or tap on category 'C' for 'RefreshView'.
3.Click or tap on test 'C6' or 'C7'.
4. Pull down to initiate a refresh.

Expected outcome:
C6: Progress spinner does not appear.
C7: The refresh spinner arrow appears, just like refresh on Android Emulator.
image

Actual outcome:
C6: No response from the page.
C7: The refresh spinner arrow doesn't appear, no response from the page.
image

Link to public reproduction project repository

https://devdiv.visualstudio.com/Pkgs/_git/VS.TestAssets.Xaml?path=/Assets/Projects/ManualMauiTests

Version with bug

Microsoft.Maui.Controls 8.0.0-nightly.9028+sha.8bf593a0b6-azdo.8295159

Is this a regression from previous behavior?

This bug is not a regression, it can be reproduced in the latest stable release: 8.0.0-preview.7.8842

Affected platforms

Windows

Affected platform versions

net8.0-windows10.0.19041.0

@v-yanfw v-yanfw added the t/bug Something isn't working label Aug 24, 2023
@v-yanfw v-yanfw changed the title [MAUI] RefreshView C7 Pull down in the purple view isn't activate the RefreshView on Surface when refreshing. [MAUI] RefreshView C7 Pull down in the purple view isn't activate the RefreshView on Surface. Aug 24, 2023
@rachelkang
Copy link
Member

Hi, @v-yanfw - thank you for reporting this issue. Are you testing this on a Windows touch device like a Windows Surface book? Or are you doing the pull to refresh via mouse?

@rachelkang rachelkang added the s/needs-info Issue needs more info from the author label Aug 24, 2023
@ghost
Copy link

ghost commented Aug 24, 2023

Hi @v-yanfw. We have added the "s/needs-info" label to this issue, which indicates that we have an open question for you before we can take further action. This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@Eilon Eilon added the area-controls-refreshview RefreshView label Aug 24, 2023
@ghost ghost added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Aug 24, 2023
@v-yanfw
Copy link
Collaborator Author

v-yanfw commented Aug 25, 2023

@rachelkang Testing this on a Windows touch device: Windows Surface book.
It's sliding the touch screen (purple view) down with finger.

Device System About Information:
Processor Microsoft SQ2 @ 3.15 GHz 3.15 GHz
Installed RAM 16.0 GB (15.5 GB usable)
System type 64-bit operating system, ARM-based processor
Pen and touch Pen and touch support with 10 touch points

@ghost ghost added s/needs-attention Issue has more information and needs another look and removed s/needs-info Issue needs more info from the author labels Aug 25, 2023
@rmarinho rmarinho added this to the Backlog milestone Aug 25, 2023
@ghost
Copy link

ghost commented Aug 25, 2023

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.

@rmarinho
Copy link
Member

So i wonder if this needs to be address soon so this manual test is green! @PureWeen @rachelkang ?

@v-yanfw v-yanfw changed the title [MAUI] RefreshView C7 Pull down in the purple view isn't activate the RefreshView on Surface. [MAUI] RefreshView C6 or C7 Pull down in the view isn't activate the RefreshView on Surface. Jan 16, 2024
@kevinxufei
Copy link
Collaborator

Verified this issue with Visual Studio Enterprise 17.10.0 Preview 1 and Microsoft.Maui.Controls version is 8.0.6.
Can repro on windows platform with repro steps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-refreshview RefreshView csi-new platform/windows 🪟 s/needs-attention Issue has more information and needs another look s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working testing-manual
Projects
None yet
Development

No branches or pull requests

8 participants