brutella / chatheads

An implementation of Facebook's ChatHeads on iOS.
ISC License
877 stars 132 forks source link

Bug: Chat Head is selected and background is touchable and not dimmed #4

Closed pedrovieira closed 11 years ago

pedrovieira commented 11 years ago

Here's the video preview to make it easier for you to see what's wrong: http://cl.ly/3B1s2r1j0f2O Steps to reproduce:

Thanks!

michaelsayman commented 11 years ago

For me at least, what I did was comment out the part where it removes the background from superview and where it makes it nil. Commenting out those two lines fixed it for me.

if (finished) { // [_backgroundView removeFromSuperview]; // _backgroundView = nil; completionBlock(); }

Update: Btw, this is located in the "Dragging" view.

elfenlaid commented 11 years ago

Also, if you drag notification screen(down swipe from status bar, widget with weather, dow jones and etc.) with ChatHead and then hide notifications screen the ChatHead would stay at the center of screen.

brutella commented 11 years ago

Should be fixed now by @elfenlaid