search

flipkart-incubator / springy-heads

Chat heads library for android
Apache License 2.0
370 stars 108 forks source link

Fix last chathead remove exception #35

Open cazador4 opened 7 years ago

TheCrafter commented 7 years ago

I just gave his PR a try and it seems to be working. He fixed the exception, that's for sure. I think you should accept it.

icoolguy1995 commented 7 years ago

@TheCrafter I've tried this fix, but when removing the last chathead, key does seems to be removed. Can you check?

TheCrafter commented 7 years ago

@icoolguy1995 I do not understand what your problem is. Could you show me your code so I can understand better?

icoolguy1995 commented 7 years ago

In a nutshell, removeView does not get called(for me) when removing last chathead.

TheCrafter commented 7 years ago

I don't know what to tell you.. I gets called for me.