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

Crash while setting the delegate: Acessibility? #41

Closed
acf opened this issue Oct 6, 2014 · 6 comments
Closed

Crash while setting the delegate: Acessibility? #41

acf opened this issue Oct 6, 2014 · 6 comments

Comments

@acf
Copy link

acf commented Oct 6, 2014

Hi,

I've just gone live with an app using the excellent PHFComposeBarView. I've seen a few crashes over the last couple of days when setting the delegate. I'm suspecting something to do with Accessibility (due both to the word appearing in the crash report, and the fact that it seems to be consistently affecting 2 users, but no-one else). It's iOS8 only, and affecting both iPhone 5S and 6+.

The crash report is here: http://crashes.to/s/0321516c355 but I've screens hotted the salient piece here:
2014-10-06_1109

I'm not expecting you to fix my bugs :-) But I just wondered if anyone else had seen it and/or figured it out.

Thanks again for a really useful component.

Alan

@acf
Copy link
Author

acf commented Oct 6, 2014

Confirmed in Xcode. It crashes as above when VoiceOver is on.

@acf
Copy link
Author

acf commented Oct 6, 2014

I fixed it by removing the Delegate Chain. I set the compose bar's delegate to my controller and the textviews delegate to the compose bar. My controller wasn't interested in textview delegate messages anyway. I still can't figure out what was actually crashing, and why the accessibility call would crash instead of just getting passed through.

@hummer330
Copy link

Thanks Alan, I had exactly same crash and spent a lot time try to figure out what was the problem. You post saves my life. Cheers

stasix added a commit to stasix/PHFComposeBarView that referenced this issue Nov 12, 2014
@jlindenbaum
Copy link

+1 Would love to see this merged into master.

@leozc
Copy link

leozc commented Dec 28, 2014

The patch works, confirmed from IPhone 6 with voice over enabled.
stasix@440e210

@zsk425
Copy link

zsk425 commented Jan 31, 2015

+1

fphilipe added a commit that referenced this issue Feb 1, 2017
@fphilipe fphilipe closed this as completed Feb 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants