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

Not able to reorder list with ion-checkbox in the list item #7339

Closed
jntr06 opened this issue Jul 16, 2016 · 6 comments
Closed

Not able to reorder list with ion-checkbox in the list item #7339

jntr06 opened this issue Jul 16, 2016 · 6 comments
Assignees
Milestone

Comments

@jntr06
Copy link

jntr06 commented Jul 16, 2016

If I put an ion-checkbox in the list item, I can't reorder it.

@manucorporat
Copy link
Contributor

@jntr06 can you provide a minimum plunker example? You can use this as a base:
http://plnkr.co/edit/me3Uk0GKWVRhZWU0usad?p=preview

also, make sure to follow the issue template. please include the output of ionic info

Thanks!

@manucorporat manucorporat added the needs: reply the issue needs a response from the user label Jul 16, 2016
@manucorporat manucorporat self-assigned this Jul 16, 2016
@jntr06
Copy link
Author

jntr06 commented Jul 17, 2016

Hi manucorporat,
Please refer to the plunker link below.
I have added sample code for the issue. Not sure why reorder is not working in plunker.
Issue is even if the reorder is working fine in my system but it is not working any more as soon as I am including a check box in the list.
Can you please help

home.html

 <ion-list reorder = "true">
   <ion-item *ngFor ="let item of items" >
   <ion-checkbox></ion-checkbox>
   <ion-label>{{item}}</ion-label>
   </ion-item>
 </ion-list>

home.ts

export class HomePage {
  greeting: string;
  constructor(private nav: NavController) {
        this.initialize();      
    }
initialize(){     
      this.items = ['USA','France','Italy','Australia','England']
    }
    goToPage1() {
      this.nav.push(Page1);
    }
}

http://plnkr.co/edit/iluWZWmibT6FSNi8K72B?p=preview

@Ionitron Ionitron removed the needs: reply the issue needs a response from the user label Jul 17, 2016
@jntr06
Copy link
Author

jntr06 commented Jul 17, 2016

@manucorporat
Copy link
Contributor

@jntr06 thanks a lot for the plunker, it helps a lot. I will look into it tomorrow!

@manucorporat manucorporat added this to the 2.0.0-beta.11 milestone Jul 18, 2016
@manucorporat
Copy link
Contributor

manucorporat commented Jul 18, 2016

@jntr06 fixed! it should work in next release! thanks for the report!

jul-18-2016 16-51-43

@jntr06
Copy link
Author

jntr06 commented Jul 18, 2016

Thank you :)

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants