Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Bug] [UWP] ScrollView issue within RefreshView #11326

Open
LakshmiNatarajan21 opened this issue Jul 6, 2020 · 6 comments
Open

[Bug] [UWP] ScrollView issue within RefreshView #11326

LakshmiNatarajan21 opened this issue Jul 6, 2020 · 6 comments

Comments

@LakshmiNatarajan21
Copy link

Description

The Entry focus is not updated properly when added inside ScrollView which loaded inside RefreshView. The issue occurs from XF 4.4. (Not sure about the exact workable version). The same working fine in Android and not tested in iOS.

This issue is the same mentioned in the below report,
#11106

The same issue occurs if we add ScrollView inside the RefreshView.

Steps to Reproduce

  1. Run the sample in UWP.
  2. Scroll down.
  3. Tap on entry.
    Note: Entry does not get the focus.

Expected Behavior

Entry should get the focus on clicking.

Actual Behavior

Entry does not get the focus.

Basic Information

  • Version with issue:
  • Last known good version:
  • IDE:
  • Platform Target Frameworks:
    • iOS:
    • Android:
    • UWP: Yes
  • Android Support Library Version:
  • Nuget Packages:
  • Affected Devices:

Reproduction Link

IssueSample

Workaround

NA

@LakshmiNatarajan21 LakshmiNatarajan21 added s/unverified New report that has yet to be verified t/bug 🐛 labels Jul 6, 2020
@PureWeen
Copy link
Contributor

PureWeen commented Jul 8, 2020

@LakshmiNatarajan21 are you still seeing this issue with the latest XF?

https://www.nuget.org/packages/Xamarin.Forms/

@PureWeen PureWeen added the s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. label Jul 8, 2020
@LakshmiNatarajan21
Copy link
Author

@PureWeen , Yes, the same issue also occurred in the latest version 4.7.0.1080.

@PureWeen PureWeen removed s/needs-info ❓ A question has been asked that requires an answer before work can continue on this issue. s/unverified New report that has yet to be verified labels Jul 10, 2020
@bmacombe
Copy link
Contributor

It's always works for whatever entries are displayed without scrolling, so even if you resize the window, entries that previously worked before they were below the scroll, then don't once they move off-screen and you scroll to them

@bmacombe
Copy link
Contributor

I tried to replicate in a standard UWP app, seems to work fine there.
image

@samhouts samhouts added this to the 5.0.0 milestone Aug 13, 2020
@AutumnEvans418
Copy link

In case this helps someone else until this gets fixed, this is what we did to get around this issue:

<Grid.Resources>
    <ScrollView x:Key="mainUI" />
</Grid.Resources>

<OnPlatform x:TypeArguments="View">
            <On Value="{StaticResource mainUI}" Platform="UWP"/>
            <OnPlatform.Default>
                <RefreshView IsRefreshing="{Binding IsBusy}" Command="{Binding RefreshCommand}" Content="{StaticResource mainUI}"/>
            </OnPlatform.Default>
</OnPlatform>

This is removes the RefreshView when running on UWP.

@samhouts samhouts removed this from the 5.0.0 milestone Nov 2, 2020
@javanjoel
Copy link

javanjoel commented Mar 18, 2021

Thanks @chrisevans9629 ... It's ashamed that we have to even do these workarounds in Xamarin. Seems like 80% of my time in Xamarin is figuring out workarounds for all these quirks.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants