-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Deprecate PullToRefreshListView control #1946
Comments
@kbrons is working on this |
PR merged |
@nmetulev If you use a PullToRefreshListView in an UWP project (min build 15063 (Creators Update), target build 17134 (W10 April Update)), you see warnings because of this deprecation:
Since you refuse to ship any updates beyond 15063 to W10M, UWP developers who don't want to exclude smartphones are stuck with that minimum version. What are we supposed to do with the PullToRefreshListView in the future? Are you serious about removing it? |
We are. You can still continue using the PullToRefreshListView by using the source directly (you can find it in our releases), but we will remove it from the nuget packages in the future and we will not fix bugs |
Why MS does not consider about our developers' feelings. Change is a good thing, but always change must be bad. |
@hupo376787, it doesn't make sense to support multiple versions of the same functionality. In this case, with the upcoming release of the Windows UI library (which also works on older versions of Windows 10), you can use the RefreshContainer in place of the PullToRefreshListView and we even have a guide on how to do it. |
@nmetulev does "older versions" include w10 mobile? |
Not sure what their minimum version is, but if it works on Creators Update, then yes. |
|
Following this guide, getting-started. And according to my test, RefreshContainer can't work on win 10 mobile 15063. I don't have other versions, so the result is not known.
|
First, you are not using the control from the nuget at all because you are not setting the target version correctly:
However, even if you do so, the project will throw the following error:
@nmetulev, could you please let the team know? |
@hupo376787, no, you cannot just target 16299... If you target 16299, it is still going to use the control in the SDK, NOT the one from the nuget package. You need to make sure the target version is at least 17134. It will still not work and this is another issue though. We are already looking into this. |
Once the platform’s native pull-to-refresh control is released, the PullToRefreshListView control will need to be deprecated and leverage the new platform control whenever possible.
We should attempt to add a new property called
UseRefreshContainerWhenPossible
to enable use of the platform control on devices where it is available.The text was updated successfully, but these errors were encountered: