UICollectionView button

Question

I'm trying to create UICollectionView with buttons. I can set button titles but target action doesn't work. Below is my code:

    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "TVButtonCell", for: indexPath) as! TVButtonCell
    cell.channelButton.setTitle(channelKeys[indexPath.row], for: .normal)
    cell.channelButton.addTarget(self, action: #selector(FirstViewController.pressed(_:)), for: .primaryActionTriggered)
    return cell
}

    func pressed(_ sender: AnyObject) {
    print("Button pressed")
}

Does anybody know what am I doing wrong?

does your app crash when you hit button? Can you past the complete stack trace? — java_doctor_101, Dec 08 '16 at 20:50
I would recommend adding the IBActions in the UICollectionView cell and having delegates in the ViewController — Nathaniel, Dec 08 '16 at 20:51
Alternatively, maybe this will help you: http://stackoverflow.com/questions/29610316/how-to-add-a-delete-button-to-collection-view-cell-in-swift?rq=1 — Nathaniel, Dec 08 '16 at 20:51

score 1 · Accepted Answer · answered Dec 08 '16 at 20:59

1

Damn, it's needed to add following func:

    func collectionView(_ collectionView: UICollectionView, canFocusItemAt indexPath: IndexPath) -> Bool {
    return false
}

answered Dec 08 '16 at 20:59

BobC

639
5
19

score 0 · Answer 2 · answered Dec 08 '16 at 20:58

you should use delegates to call methods on viewcontrollers.

protocol TVButtonCellDelegate: class {
    func buttonClicked()
}

then create weak variable on TVButtonCell to pass delegate.

weak var delegate: TVButtonCellDelegate?

and also IBAction on tvbuttoncell to call method inside delegate

@IBAction func tvButtonClicked(sender: UIBUtton) {
   delegate?. buttonClicked()
}

then you simply apply delegate on firstviewcontroller

extension FirstViewController: TVButtonCellDelegate {
   func buttonClicked() {
       print("button clicked")
   }
}

and then assign firstcontroller to cell when you create one

 func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
    let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "TVButtonCell", for: indexPath) as! TVButtonCell
    cell.channelButton.setTitle(channelKeys[indexPath.row], for: .normal)
    cell.delegate = self
    return cell

UICollectionView button

2 Answers2