Set myself as popupBarLongPressGestureRecognizerDelegate about lnpopupcontroller

[funkenstrahlen]

I tried this:

        let longpressRecognizer = UILongPressGestureRecognizer(target: self, action: "handleLongPress:")
        self.popupBar.addGestureRecognizer(longpressRecognizer)

    func handleLongPress(recognizer: UITapGestureRecognizer) {
        print("long press")
    }

But the action does not get called.

[LeoNatan]

What are you attempting to do? Listen to long press on the bar?

[funkenstrahlen]

If the popup controller is minified I want to listen to a long press on the whole visible player bar.

On 04.12.2015, at 17:57, Leo Natan [email protected] wrote:

What are you attempting to do? Listen to long press on the bar?

—
Reply to this email directly or view it on GitHub.

[LeoNatan]

The bar already includes a long press gesture recognizer. So with your own recognizer, set the delegate and implement the gestureRecognizer(_, shouldRecognizeSimultaneouslyWithGestureRecognizer:) delegate method. Return true in this method to allow simultaneous recognition.

[funkenstrahlen]

Thank you for pointing me to the problem with multiple gesture recognizers. This helps a lot.

However I am currently struggling with the first part: Setting the delegate.

popupBarLongPressGestureRecognizerDelegate is an attribute of LNPopupController. But this is all in the private part of your code, so I can not access it.

What about defining a delegate protocol for the whole lnpopup project, so I can react to certain events and making this a real public api?

[LeoNatan]

You should not modify the internals of LNPopupController. Instead, add an additional long press gesture recognizer (as you have in the example above), and set that recognizer's delegate.

[funkenstrahlen]

Ok I did that:

    override func viewDidLoad() {
        super.viewDidLoad()
        let longpressRecognizer = UILongPressGestureRecognizer(target: self, action: "handleLongPress:")
        longpressRecognizer.delegate = self
        self.popupBar.addGestureRecognizer(longpressRecognizer)
    }

    func handleLongPress(recognizer: UILongPressGestureRecognizer) {
        print("long press")
    }

    func gestureRecognizer(gestureRecognizer: UIGestureRecognizer, shouldRecognizeSimultaneouslyWithGestureRecognizer otherGestureRecognizer: UIGestureRecognizer) -> Bool {
        return true
    }

However I still do not receive the long press event.

Is that the way you intended to tell me?

[funkenstrahlen]

Apparently the problem seems to be that self.popupBar is not valid.

[LeoNatan]

Just did the following in the demo project and it worked.

    func longPress(gestureRecognizer: UILongPressGestureRecognizer) {
        print("Are you talking to me?")
    }

    func gestureRecognizer(gestureRecognizer: UIGestureRecognizer, shouldRecognizeSimultaneouslyWithGestureRecognizer otherGestureRecognizer: UIGestureRecognizer) -> Bool
    {
        return true
    }

    override func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) {
        let popupContentController = storyboard?.instantiateViewControllerWithIdentifier("DemoMusicPlayerController") as! DemoMusicPlayerController
        popupContentController.songTitle = titles[indexPath.row]
        popupContentController.albumTitle = subtitles[indexPath.row]

        tabBarController?.presentPopupBarWithContentViewController(popupContentController, animated: true, completion: nil)

        let lpgr = UILongPressGestureRecognizer(target: self, action: "longPress:")
        lpgr.delegate = self
        print(tabBarController?.popupBar)
        tabBarController?.popupBar?.addGestureRecognizer(lpgr)

        tableView.deselectRowAtIndexPath(indexPath, animated: true)
    }

Remember the popupBar property is non-nil only after presenting the popup. I did find a bug in the header file, where the popupBar is not marked as nullable and will push a fix.

[funkenstrahlen]

I got it working now too!

I tried to configure self.popupBar in viewDidLoad() of popupContentController but popupBar was not valid yet. I did it now in prepareForSegue and it works now.

[LeoNatan]

👍

Make sure to pull the latest changes.

Thanks,
Leo Natan

On Dec 4, 2015, at 23:05, Stefan Trauth [email protected] wrote:

I got it working now too!

I tried to configure self.popupBar in viewDidLoad() of popupContentController but popupBar was not valid yet. I did it now in prepareForSegue and it works now.

—
Reply to this email directly or view it on GitHub.

[funkenstrahlen]

Thanks a lot for helping me! ❤️

[LeoNatan]

Cheers! Happy coding!