sammcewan / WYPopoverController

WYPopoverController is for the presentation of content in popover on iPhone / iPad devices. Very customizable.
Other
253 stars 74 forks source link

Popover not re-appearing after calling dismissPopoverAnimated #54

Open boazin opened 9 years ago

boazin commented 9 years ago

On iOS7 only (iOS8 it's OK) - when manually doing [popoverController dismissPopoverAnimated:YES];

And then trying to re-open the popover (using presentPopoverFromRect)

In the original project I got observation leak. On this fork (thanks for maintaining BTW) I don't get an error but I'm experiencing the same issue.

The error I got when used the original project:

An instance 0x7fa658d31340 of class NewChapterViewController was deallocated while key value observers were still registered with it. Observation info was leaked, and may even become mistakenly attached to some other object. Set a breakpoint on NSKVODeallocateBreak to stop here in the debugger. Here's the current observation info:
<NSKeyValueObservationInfo 0x7fa65b01eda0> (
<NSKeyValueObservance 0x7fa658ca31d0: Observer: 0x7fa658d33220, Key path: preferredContentSize, Options: <New: NO, Old: NO, Prior: NO> Context: 0x0, Property: 0x7fa658c829f0>

Tried to debug it - it seems that this is the preferredContentSize but this is as far as I succeeded

boazin commented 9 years ago

I can confirm that this issue appears when the popover is originating from a tableviewcell. If the cell is out of screen and gets reused, it is OK

sammcewan commented 9 years ago

Have you got an example project of it doing this? It's a pretty specific scenario.