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

Windows adds a range of items to a CollectionView in reverse #22096

Open
KeithBoynton opened this issue Apr 27, 2024 · 3 comments
Open

Windows adds a range of items to a CollectionView in reverse #22096

KeithBoynton opened this issue Apr 27, 2024 · 3 comments
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/windows 🪟 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working

Comments

@KeithBoynton
Copy link

Description

When adding a range of items, say a string list

new List<string>() { "One", "Two", "Three", "Four", "Five" }

to a CollectionView via an event on Windows it adds it in reverse whereas on iOS, MacCatalyst and Android it add them in the correct order.

Steps to Reproduce

  1. Clone the attached project
  2. Run the solution on MacCatalyst
  3. Click the Add button and observe the items in the correct order
  4. Run the solution on iOS
  5. Click the Add button and observe the items in the correct order
  6. Run the solution on Android
  7. Click the Add button and observe the items in the correct order
  8. Run the solution on Windows
  9. Click the Add button and observe the items in the incorrect REVERSE order

Link to public reproduction project repository

https://bitbucket.org/KeithBoynton/collection-view-reverse/src/master/

Version with bug

8.0.21 SR4.1

Is this a regression from previous behavior?

Not sure, did not test other versions

Last version that worked well

Unknown/Other

Affected platforms

Windows

Affected platform versions

No response

Did you find any workaround?

No

Relevant log output

No response

@KeithBoynton KeithBoynton added the t/bug Something isn't working label Apr 27, 2024
@Zhanglirong-Winnie Zhanglirong-Winnie added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Apr 28, 2024
@Zhanglirong-Winnie
Copy link
Collaborator

Verified this issue with Visual Studio 17.10.0 Preview 5(8.0.21&8.0.14&8.0.3). Can repro on windows platforms with sample project.
Android:
image
Windows:
image

@Eilon Eilon added the area-controls-collectionview CollectionView, CarouselView, IndicatorView label Apr 30, 2024
@wyyqyl
Copy link

wyyqyl commented May 2, 2024

#if WINDOWS
            changedItems.Reverse();
#endif

@KeithBoynton
Copy link
Author

#if WINDOWS
            changedItems.Reverse();
#endif

Lets not do that, it's a bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-collectionview CollectionView, CarouselView, IndicatorView platform/windows 🪟 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants