53

So I have a main object that has many images associated with it. An Image is also an object.

Say you have a collection view controller, and in that controller you have

cellForItemAtIndexPath

well based on the main object, if it has the current image associated with it I want to set selected to true. But I want the user to be able to "un-select" that current cell at any time to remove its association with the main object.

I find that if you set "selected to true" - if there is an relation between the main object and image in cellForItemAtIndexPath, de-selection is no longer an option.

in

didDeselectItemAtIndexPath

and

didSelectItemAtIndexPath

I test with a log to see if they are called. If a cell is set to selected - nether gets called, but If I never set a cell to selected in cellForItemAtIndexPath I can select and deselect all I want.

Is this the intended way a collection view is supposed to work? I read the docs and it does not seem to talk about this being so. I interpret the docs to mean it works the way a table view cell would. with a few obvious changes

This also shows the controller is set up correct and is using the appropriate delegate methods.... hmmmm

0

15 Answers 15

93

I had the same issue, ie. setting cell.selected = YES in [UICollectionView collectionView:cellForItemAtIndexPath:] then not being able to deselect the cell by tapping on it.

Solution for now: I call both [UICollectionViewCell setSelected:] and [UICollectionView selectItemAtIndexPath:animated:scrollPosition:] in [UICollectionView collectionView:cellForItemAtIndexPath:].

10
  • 2
    Strange that it should be necessary - but it works.. And it is not causing infinite loops with didSelectItemAtIntexPath delegate.. Commented Jul 30, 2013 at 9:58
  • 6
    Has anyone found a better solution since this was proposed? This solution also works for me, but this might be a bug. Commented Feb 20, 2014 at 20:32
  • 4
    You are a SAINT! A SAINT I TELL YOU! Not sure on the specific of the ScrollPosition enumeration but its working. (Y)
    – MrOli3000
    Commented Sep 5, 2014 at 11:00
  • This is perhaps due to the internal way in which UICollectionView manages a reference on selections? It seems you need to call setSelected: to reflect on the view level too. Whatever, don't care, this works.
    – Adam Waite
    Commented Sep 24, 2014 at 13:53
  • 1
    Cheers to you! I think it was needing to call [UICollectionView selectItemAtIndexPath:animated:scrollPosition:]. Thanks!
    – Alex Sharp
    Commented May 21, 2015 at 20:13
48

I had a Deselect issue with UICollectionView and I found that I was not allowing multiple selection on collectionView. So when I was testing I tried always on the same cell and if single selection is ON you can't Deselect a cell already selected.

I had to add:

myCollectionView.allowsMultipleSelection = YES;
1
  • I guess the reason most people miss that, is because it's not exposed in IB for collection views, right? I've looked for it, couldn't find it anywhere in IB.
    – Sagi Mann
    Commented Dec 21, 2016 at 12:12
46

Do you have a custom setSelected method in your Cell class? Are you calling [super setSelected:selected] in that method?

I had a mysterious problem where, I was using multiple selection, I could not deselect cells once they were selected. Calling the super method fixed the problem.

2
  • 8
    I cannot believe that I missed something so obvious after 5 years of hard iOS programming. Duh!
    – Linasses
    Commented Nov 19, 2013 at 12:56
  • 2
    this can be easily missed Commented Feb 25, 2016 at 17:29
45

This is kind of old but, since I encounter the same issue using swift I will add my answer. When using:

 collectionView.selectItemAtIndexPath(indexPath, animated: true, scrollPosition: [])

The cell didn't get selected at all. But when using:

cell.selected = true

It did get selected but I wasn't able to select/deselect the cell anymore.

My solution (use both methods):

cell.selected = true
collectionView.selectItemAtIndexPath(indexPath, animated: true, scrollPosition: .None)

When this two methods are called in:

func collectionView(collectionView: UICollectionView, cellForItemAtIndexPath indexPath: NSIndexPath) -> UICollectionViewCell

It worked perfectly!

6
  • This fixex my problem. Thanks a lot.
    – Ismail
    Commented Nov 16, 2015 at 12:04
  • Other options didn't work for me, this one worked perfectly.
    – Stuart
    Commented Aug 3, 2016 at 8:51
  • perfect answer even three and half year later !
    – oguzhan00
    Commented Nov 30, 2018 at 8:52
  • .None isn't a valid scroll position ?
    – tennis779
    Commented Mar 13, 2019 at 18:24
  • 4 years later and here we are. Was having issues deselecting cells as well.
    – cumanzor
    Commented Sep 13, 2019 at 21:36
10

I don't know why UICollectionView is so messy like this compared to UITableViewController... A few things I found out.

The reason why setSelected: gets called multiple times is because of the sequence methods get called. The sequence is very similar to that of UITextFieldDelegate methods.

The method collectionView:shouldSelectItemAtIndexPath: is called before the collectionView actually selects the cell because it's actually asking "should it be selected"?

collectionView:didSelectItemAtIndexPath: is in fact called after the collectionView selects the cell. Hence the name "did select."

So this is what is happening in your case (and my case, and I had to wrestle hours over this).

A selected cell is touched again by the user to deselect. shouldSelectItemAtIndexPath: is called to check whether the cell should be selected. The collectionView selects the cell and then didSelectItemAtIndexPath is called. Whatever you do at this point is after the the cell's selected property is set to YES. That's why something like cell.selected = !cell.selected won't work.

TL;DR - Have your collectionView deselect the cell in the delegate method collectionView:shouldSelectItemAtIndexPath: by calling deselectItemAtIndexPath:animated: and return NO.

Short example of what I did:

- (BOOL)collectionView:(OPTXListView *)collectionView shouldSelectItemAtIndexPath:(NSIndexPath *)indexPath {
    NSArray *selectedItemIndexPaths = [collectionView indexPathsForSelectedItems];

    if ([selectedItemIndexPaths count]) {
        NSIndexPath *selectedIndexPath = selectedItemIndexPaths[0];

        if ([selectedIndexPath isEqual:indexPath]) {
            [collectionView deselectItemAtIndexPath:indexPath animated:YES];

            return NO;
        } else {
            [collectionView selectItemAtIndexPath:indexPath animated:YES scrollPosition:UICollectionViewScrollPositionCenteredHorizontally];

            return YES;
        }
    } else {
        [collectionView selectItemAtIndexPath:indexPath animated:YES scrollPosition:UICollectionViewScrollPositionCenteredHorizontally];

        return YES;
    }
}
1
  • Unique and right answer. Without set collectionView.allowsMultipleSelection = YES;
    – WINSergey
    Commented Oct 21, 2016 at 8:31
7

Here is my answer for Swift 2.0.

I was able to set the following in viewDidLoad()

collectionView.allowsMultipleSelection = true;

then I implemented these methods

func collectionView(collectionView: UICollectionView, didSelectItemAtIndexPath indexPath: NSIndexPath) {
    let cell = collectionView.cellForItemAtIndexPath(indexPath) as! MyCell
    cell.toggleSelected()
}

func collectionView(collectionView: UICollectionView, didDeselectItemAtIndexPath indexPath: NSIndexPath) {
    let cell = collectionView.cellForItemAtIndexPath(indexPath) as! MyCell
    cell.toggleSelected()
}

finally

class MyCell : UICollectionViewCell {

    ....

    func toggleSelected ()
    {
        if (selected){
            backgroundColor = UIColor.orangeColor()
        }else {
            backgroundColor = UIColor.whiteColor()
        }
    }

}
2
  • This works, but i need to change the border color of the selected cell, that doesn't work in the cell class. Can you suggest anything on that? Commented Dec 20, 2016 at 13:00
  • @ArpitDhamane This doesn't work for me either and I'm using Swift 3 Xcode 8.1. I can only change cell style inside "collectionView: UICollectionView, cellForItemAt indexPath:"
    – KMC
    Commented Apr 22, 2017 at 18:34
6

This question is six years old, but I don't care; after landing here and finding that none of the answers solved my specific problem, I eventually hit upon what I consider to be the best answer to this question. So, I'm posting it here for posterity.

Selecting cells in cellForItemAtIndexPath is problematic because of the way UICollectionView calls that method. It's not just called when initially setting up a collection view. It's called continuously as the collection view is scrolled, because the collection view only ever asks its data source for visible cells, thereby saving a lot of overhead.

Because collection views don't keep all of their cells in memory, they need to manage the selected state of their own cells. They don't expect you to provide them with cells whose isSelected property has been set. They expect you to provide them with cells, and they will set the selected property on them if appropriate.

This is why Apple cautions you to not set the isSelected property of UICollectionViewCell directly. UICollectionView expects to take care of that for you.

SO, the answer is to NOT attempt to select cells in the cellForItemAtIndexPath method. The best place to select cells that you want to be initially selected is in the viewWillAppear method of the UICollectionViewController. In that method, select all desired cells by calling UICollectionView.selectItem(at:animated:scrollPosition:), and DON'T set isSelected directly on your cells.

2
 func collectionView(collectionView: UICollectionView, didSelectItemAtIndexPath indexPath: NSIndexPath) {

    let cell = collectionView.cellForItemAtIndexPath(indexPath)
    if cell?.selected == true{
        cell?.layer.borderWidth = 4.0
         cell?.layer.borderColor = UIColor.greenColor().CGColor
    }   
}func collectionView(collectionView: UICollectionView, didDeselectItemAtIndexPath indexPath: NSIndexPath) {
    let cell = collectionView.cellForItemAtIndexPath(indexPath)
    if cell?.selected == false{
            cell?.layer.borderColor = UIColor.clearColor().CGColor
    }

}

Simple Solution i found

2

I'm using a custom cell subclass, for me I just had to set self.selected = false in prepareForReuse() in the subclass.

2

Living in the age of iOS 9, there are multiple things to check here.

  1. Check do you have collectionView.allowsSelection set to YES
  2. Check do you have collectionView.allowsMultipleSelection set to YES (if you need that ability)

Now comes the fan part. If you listen to Apple and set backgroundColor on the cell.contentView instead of cell itself, then you have just hidden its selectedBackgroundView from ever being visible. Because:

(lldb) po cell.selectedBackgroundView
<UIView: 0x7fd2dae26bb0; frame = (0 0; 64 49.5); autoresize = W+H; layer = <CALayer: 0x7fd2dae26d20>>

(lldb) po cell.contentView
<UIView: 0x7fd2dae22690; frame = (0 0; 64 49.5); gestureRecognizers = <NSArray: 0x7fd2dae26500>; layer = <CALayer: 0x7fd2dae1aca0>>

(lldb) pviews cell
<MyCell: 0x7fd2dae1aa70; baseClass = UICollectionViewCell; frame = (0 0; 64 49.5); clipsToBounds = YES; hidden = YES; opaque = NO; layer = <CALayer: 0x7fd2dae1ac80>>
   | <UIView: 0x7fd2dae26bb0; frame = (0 0; 64 49.5); autoresize = W+H; layer = <CALayer: 0x7fd2dae26d20>>
   | <UIView: 0x7fd2dae22690; frame = (0 0; 64 49.5); gestureRecognizers = <NSArray: 0x7fd2dae26500>; layer = <CALayer: 0x7fd2dae1aca0>>
   |    | <UIView: 0x7fd2dae24a60; frame = (0 0; 64 49.5); clipsToBounds = YES; alpha = 0; autoresize = RM+BM; userInteractionEnabled = NO; layer = <CALayer: 0x7fd2dae1acc0>>
   |    | <UILabel: 0x7fd2dae24bd0; frame = (0 0; 64 17.5); text = '1'; opaque = NO; autoresize = RM+BM; userInteractionEnabled = NO; layer = <_UILabelLayer: 0x7fd2dae240c0>>
   |    | <UILabel: 0x7fd2dae25030; frame = (0 21.5; 64 24); text = '1,04'; opaque = NO; autoresize = RM+BM; userInteractionEnabled = NO; layer = <_UILabelLayer: 0x7fd2dae25240>>

(lldb) po cell.contentView.backgroundColor
UIDeviceRGBColorSpace 0.4 0.4 0.4 1

So if you want to use selectedBackgroundView (which is the one being turned on/off with cell.selected and selectItemAtIndexPath...) then do this:

cell.backgroundColor = SOME_COLOR;
cell.contentView.backgroundColor = [UIColor clearColor];

and it should work just fine.

3
  • 1
    Where do you get pviews from? I get an error: Unrecognized command 'pview'. on Xcode 7.2.1, lldb-340.4.119.1 Commented Apr 25, 2016 at 18:15
  • 2
    pviews cell is simply a shortcut for po [cell recursiveDescription] call, which you can do on any view in lldb console Commented Apr 26, 2016 at 7:31
  • correct. in order to use selected background view on collection view cell, only set color on cell background color, not the content view.
    – Yllow
    Commented Mar 16, 2017 at 8:40
1

I don't know that I understand the problem, but selected status is set per cell and would include all subviews within the cell. You don't explain what you mean by "a main object has many images associated with it." Associated as in subviews? Or what kind of association exactly do you mean?

It sounds like a design problem to me. Perhaps you need a UIView subclass that contains whatever associated objects you need to have; that subclass can then be set as the content view. I do this, for example, where I have an image, a description, and a sound recording related to the image. All are defined in the subclass and then each of these subclasses becomes a content view for a single cell.

I've also used an arrangement to related images to a folder which contains them. Under this set up, folders and images each have a subclass and either one might be attached to a cell as a content view (these are stored in core data as a single entity).

Perhaps you can further explain your problem?

7
  • 1
    has anyone used the collection view, set selected to YES in cellForItemAtIndexPath and was able to de-select that cell? I have set up two test projects and both work the same. As soon as you set selected to true in cellForItemAtIndexPath deselect is never called again.
    – bworby
    Commented Mar 12, 2013 at 4:46
  • No, I don't think you can do that. If I set a cell to selected in celLForItemAtIndexPath, the item(s) behave as selected from the initial display of the collection view (i.e., selected background); however, if I look at the value of [collectionView indexPathsForSelectedItems], I get nil -- which suggests to me there would not be a way of deselecting it through the usual means. You may be able to set the value of cell.selected == NO, as I didn't try that. But if you can't get the index path of selected items, that is definitely going to make the collection view harder to work with. Commented Mar 12, 2013 at 5:03
  • does this not seem odd? I may be thinking about it wrong but I could definitely see a ton of uses for allowing certain cells to start as selected, and be able to be deselected.
    – bworby
    Commented Mar 13, 2013 at 21:55
  • It does seem odd, but there may be some reason for it. Have you tried using selectItemAtIndexPath: ? Commented Mar 14, 2013 at 3:49
  • Yes, I can get it to work that way or highlighting a cell and not selecting it. But then the fist touch does nothing (because that is the initial selection) and then it will deselect and re-select.
    – bworby
    Commented Mar 15, 2013 at 4:46
1

Have you looked at:

- (BOOL)collectionView:(PSTCollectionView *)collectionView shouldDeselectItemAtIndexPath:(NSIndexPath *)indexPath;

Please state your problem more clearly and perhaps we can get to the bottom of it. I've just spent some time with UICollectionView.

I think your problem may be stemming from the confusion that if you set cell.selected = YES programmatically, the reason didSelectItemAtIndexPath: is not getting called is because that is only used when the collectionView itself is responsible for the cell's selection (eg. via a tap).

1

When calling both [UICollectionViewCell setSelected:] and [UICollectionView selectItemAtIndexPath:animated:scrollPosition:] in [UICollectionView collectionView:cellForItemAtIndexPath:] doesn't work try calling them inside a dispatch_async(dispatch_get_main_queue(), ^{}); block.

That's what finally fixed it for me.

0
1

Collectionview select, deselect issue solved, when I did this. In viewDidLoad, add collViewRiskPreferences.allowsMultipleSelection = false and add the following

func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {

        guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier:"MyCell", for: indexPath) as? MyCell else { return UICollectionViewCell() }
        
        cell.setupCellStyle(isSelected: false)
        
        return cell
    }

func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        guard let cell = collectionView.cellForItem(at: indexPath) else { return }
        cell.setupCellStyle(isSelected: true)
    }
    
    func collectionView(_ collectionView: UICollectionView, didDeselectItemAt indexPath: IndexPath) {
        guard let cell = collectionView.cellForItem(at: indexPath) else { return }
        cell.setupCellStyle(isSelected: false)
    }

and

class MyCell: UICollectionViewCell {
        func setupCellStyle(isSelected: Bool) {
            if(isSelected) {
                self.contentView.backgroundColor = UIColor.blue
            } else {
                self.contentView.backgroundColor = UIColor.green
            }
        }
    }
-1

Cell selection and deselection is best handled by setting a backgroundView and a selected background view. I recommend making sure that both of these views' frames are set correctly in the layoutSubviews method (if you set the selected and background view via IB).

Don't forget to set your contentView's (if you have one) background color to clear so the correct background view shows through.

Never set the cell's selection directly (i.e. via cell.selected = YES), use the methods designed for this purpose in the collection view. It is clearly explained in the documents, although I will agree the information is somewhat fragmented across guides.

You should not need to poke into a cell's background colors directly in your collectionView datasource.

Also, as a final note, don't forget to call [super prepareForReuse] and [super setSelected:selected] if you are implementing these in your cell's class, as you might be preventing the cell's superclass from doing the cell selection.

Hit me up if you need further clarification on this subject.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.