hectormatos2011 / ioscontextualmenu Goto Github PK

Hi, im having trouble using this framework when launching from a URL scheme. All the necessary methods get called to setup my array, the delegate methods for returning the label and image all gets called but the menu itself never appears when launching from a URL

I'm using it for collection views cells and it seems that didSelectItemAt of the collection view is not called because the menu gesture recognizers overrides the touches.
So to get the cell indexPath where the menu is shown I have modified:

//ContextualMenu+Extension

-    @objc(contextualMenu:didSelectItemAtIndex:)
-    optional func contextualMenu(_ menu: ContextualMenu, didSelectItemAt index: Int)
+    @objc(contextualMenu:didSelectItemAtIndex:sourceAtPoint:)
+    optional func contextualMenu(_ menu: ContextualMenu, didSelectItemAt index: Int, sourceAt point: CGPoint)

and

//ContextualMenu / didSelect(menuItemView: ContextualMenuItemView)

//      delegate?.contextualMenu?(self, didSelectItemAt: menuItem.index)
        delegate?.contextualMenu?(self, didSelectItemAt: menuItem.index, sourceAt: startingLocation)

Using it this way

func contextualMenu(_ menu: ContextualMenu, didSelectItemAt index: Int, sourceAt point: CGPoint) {
let point2 = UIApplication.shared.keyWindow?.convert(point, to: collectionView)
let indexPath = collectionView.indexPathForItem(at: point2)
let cell = collectionView.cellForItem(at: indexPath)
}

Is it acceptable or is there a better way to use it with collectionViews?

In layoutMenuItemsIfNeeded the titleView is set to a custom view if the delegate responds to the custom view method. Right after that though, there is an if-condition that overrides the titleView in any case, either with the regular case or with an invisible Item (line 719), or am I wrong here.

My users are having issues with the menu freezing when force touch is (accidently) triggered. I dont have a force touch capable device to tinker with, so all i can do is inform future readers.

@hectormatos2011 tagging you in case you still aren't receiving notifications

Is there a way to present the menu programmatically?