Changing a property on nested object causes ordering problems

kryptus36 commented 4 years ago

Which platform(s) does your issue occur on?

iOS/Android/Both

Only tested on android

Please, provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it)

6.7.4

Cross-platform modules: (check the 'version' attribute in the node_modules/tns-core-modules/package.json file in your project)

6.5.1

Runtime(s): (look for the "tns-android" and "tns-ios" properties in the package.json file of your project)

Android 6.5.0

Plugin(s): (look for the version numbers in the package.json file of your project and paste your dependencies and devDependencies here)

"dependencies": { "@nativescript/theme": "^2.2.1", "nativescript-pager": "^12.0.0-alpha.8", "nativescript-vue": "^2.6.1", "tns-core-modules": "^6.5.1" }, "devDependencies": { "@babel/core": "^7.0.0", "@babel/preset-env": "^7.0.0", "babel-loader": "^8.1.0", "nativescript-dev-webpack": "^1.5.1", "nativescript-vue-template-compiler": "^2.6.0", "nativescript-worker-loader": "~0.11.0", "node-sass": "^4.13.1", "tns-platform-declarations": "^6.5.1", "typescript": "^3.8.3", "@types/node": "^13.11.1", "vue": "^2.6.11", "vue-loader": "^15.9.1" }

Please, tell us how to recreate the issue in as much detail as possible.

I've created a simple bingo card as an example. There is an array of 5 bingo cards in the pager, each has some spaces daubed and some not.

Here's the really odd part: tapping a space triggers the toggleDaub method, and it seems to work fine on cards index 0 & 1. But try it on cards 2,3,4 and the cards at the end get moved to the beginning -- even though the ordering of the array itself has not changed.

Is there any code involved?

https://github.com/kryptus36/ns-pager-ordering

triniwiz commented 4 years ago

Hi have you tried binding to a observablearray ?

kryptus36 commented 4 years ago

I just tried now (code is pushed to above repo). Unfortunately the results are the same.

triniwiz commented 4 years ago

Thanks .. can you run it on iOS if not that’s ok

kryptus36 commented 4 years ago

Ah - that's interesting. Works as expected on iOS.

kryptus36 commented 4 years ago

Just as a note, it breaks similarly on iOS if you don't use the ObservableArray.

Thanks for your quick replies!

triniwiz commented 4 years ago

@kryptus36 I gave the ObservableArray more love since it has changeEvents which made things a bit easier and more performant i'll throw something together soon

triniwiz / nativescript-pager