Repository navigation
bug: [4.6.0] ion-slide elements in wrong order when pushing a new slide to array #18633
Description
Bug Report
Ionic version:
[x] 4.6.0
Current behavior:
When dynamically pushing new slides to ion-slides it adds the new slide one place before the last slide and leaves the previous last one on the last slide even thought the printed array is in correct order.
Expected behavior:
Add pushed slide item to the end of the slides.
Steps to reproduce:
- git clone https://github.com/nerdic-coder/ion-slider-sorting-bug.git
- cd ion-slider-sorting-bug
- npm install
- npm start
- See the slide is at zero.
- Slide forward and you see a 1.
- Slide forward again and you see a 3, but it should be slide number 2 as printed in the console log.
Array is:
0: {photoId: 0}
1: {photoId: 1}
2: {photoId: 2}
3: {photoId: 3}
But elements are:
0
1
3
2
Related code:
A sample application via GitHub
https://github.com/nerdic-coder/ion-slider-sorting-bug/blob/master/src/components/app-home/app-home.tsx
Other information:
Ionic info:
PS E:\workspaces\ion-slider> ionic info
Ionic:
Ionic CLI : 5.1.1 (C:\Users\johan\AppData\Roaming\npm\node_modules\ionic)
Utility:
cordova-res : 0.3.0
native-run : not installed
System:
NodeJS : v10.15.3 (C:\Program Files\nodejs\node.exe)
npm : 6.6.0
OS : Windows 10
Activity
nerdic-coder commentedon Jun 27, 2019
Made a comparison branch with Ionic 4.5.0 and it works correct in that version:
https://github.com/nerdic-coder/ion-slider-sorting-bug/tree/4.5.0
brandyscarney commentedon Jul 1, 2019
Thanks for the issue! I am going to close this as it has been fixed in Ionic, it will be in the 4.6.1 hotfix release. Please follow #18655 for updates on this. 🙂
ionitron-bot commentedon Jul 31, 2019
Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.