search

tjerkw / Android-SlideExpandableListView

A better ExpandableListView, with animated expandable views for each list item
Apache License 2.0
1.98k stars 741 forks source link

ActionSlideExpandableListView.OnActionClickListener method name #15

Closed passsy closed 11 years ago

passsy commented 11 years ago

Hello,

The Listview works great. Thanks!

May you please change the onClick method name? I let my Fragment implement the listener. But I think an onClick method in a Fragments isn't nice to read. My first suggestion was "onClick? ... onClick WHAT?"

Something like onExpandableItemClick or so would be nicer.

Pascal

tjerkw commented 11 years ago

Hmm. I understand your confusion. But the point is that the listener is already named "OnActionClickListener". An action is a term for one of the ExpandableItems. So that's the reasoning I had behind it. Renaming it would break all my code which works with this lib.

anuran commented 11 years ago

I have an image in my list view. I want to change the image on clicking the list view item. but i am not getting where to write the code to change the image. I have used your code for expandable list view and its working fine.

tjerkw commented 11 years ago

@anuran i'm not here to help you with writing android code. Your issue is unrelated to my library.

Since im not going to rename it i'm closign this issue.

deepanraj commented 10 years ago

i need onclick listener for major list. i need to trigger major listview programmatically without user interaction and also i tried performItemClick but their no effects.