Fix crash on delete pressed for JSQMessagesComposerTextView #1664
Conversation
Current coverage is 60.97%@@ develop #1664 diff @@
==========================================
Files 62 62
Lines 2264 2265 +1
Methods 637 638 +1
Messages 0 0
Branches 137 137
==========================================
Hits 1381 1381
- Misses 811 812 +1
Partials 72 72
|
Thanks @GianniCarlo ! 👍 |
Hm, after looking at iMessage, I think we should remove |
In fact, shouldn't we just call then we wouldn't need to enumerate all those selectors.... |
you are right @jessesquires , if we remove the |
346d215
to
cfa5872
Compare
Updated pull request |
Here's what I'm suggesting: - (BOOL)canPerformAction:(SEL)action withSender:(id)sender {
[UIMenuController sharedMenuController].menuItems = nil;
return [super canPerformAction:action withSender:sender];
} wouldn't this fix the crash and keep the fix from #1281? |
oooh, hadn't seen that issue, yup then the function is indeed needed, I'll update it now |
cfa5872
to
43ce2f5
Compare
…w` to call super
updated 👍 |
Awesome. So this fixes the crash, and still fixes #1281? |
yes, fixes the crash as the default list of actions doesn't include the action |
Thanks so much for jumping on this so quickly @GianniCarlo ! 🎉 |
43ce2f5
to
8c741fd
Compare
cherry-picked into |
…w` to call super (jessesquires#1664). Fixes jessesquires#1663.
…w` to call super (jessesquires#1664). Fixes jessesquires#1663.
…iewController * 'master' of https://github.com/jessesquires/JSQMessagesViewController: (86 commits) ImageOptim on assets (jessesquires#1845) update changelog and spec for 7.3.4 Fix issue jessesquires#1583: Don't highlight cell outside message bubble (when long press) (jessesquires#1744) update changelog and version for 7.3.3 Updated `canPerformAction:withSender:` in `JSQMessagesComposerTextView` to call super (jessesquires#1664). Fixes jessesquires#1663. Update CHANGELOG and version nums for 7.3.2 fix KVO crash. close jessesquires#1631 formatting small fix regarding scrollToIndexPath (jessesquires#1642) close jessesquires#1640 Update CHANGELOG.md update CHANGLOG. bump version numbers Reverted jessesquires#1588 to fix jessesquires#1602 and fix jessesquires#1604. (jessesquires#1623) bump version numbers Update CHANGELOG.md provide default init values for JSQMessagesCollectionViewLayoutAttributes to prevent assertion. fix jessesquires#1338 follow up for jessesquires#1247 and jessesquires#1591. obfuscate private APIs. swizzle via +initialize copy attributes clean up fix keyboard hiding bug on iOS 9 (jessesquires#1307). fix jessesquires#1063 - cleanup from PR jessesquires#1281 - fix menu actions, close jessesquires#1321 - make notification methods public - partially apply changes from PR jessesquires#1563 ...
…nch3 * origin/master: (66 commits) ImageOptim on assets (jessesquires#1845) update changelog and spec for 7.3.4 Fix issue jessesquires#1583: Don't highlight cell outside message bubble (when long press) (jessesquires#1744) update changelog and version for 7.3.3 Updated `canPerformAction:withSender:` in `JSQMessagesComposerTextView` to call super (jessesquires#1664). Fixes jessesquires#1663. Update CHANGELOG and version nums for 7.3.2 fix KVO crash. close jessesquires#1631 formatting small fix regarding scrollToIndexPath (jessesquires#1642) close jessesquires#1640 Update CHANGELOG.md update CHANGLOG. bump version numbers Reverted jessesquires#1588 to fix jessesquires#1602 and fix jessesquires#1604. (jessesquires#1623) bump version numbers Update CHANGELOG.md provide default init values for JSQMessagesCollectionViewLayoutAttributes to prevent assertion. fix jessesquires#1338 follow up for jessesquires#1247 and jessesquires#1591. obfuscate private APIs. swizzle via +initialize copy attributes clean up fix keyboard hiding bug on iOS 9 (jessesquires#1307). fix jessesquires#1063 - cleanup from PR jessesquires#1281 - fix menu actions, close jessesquires#1321 - make notification methods public - partially apply changes from PR jessesquires#1563 ...
Pull request checklist
This fixes issue #1663.
What's in this pull request?
Fixes the crash on delete menu button pressed