UICollectionView Select and Deselect issue

Question

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

Answer 1

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:].

Answer 2

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;

Answer 3

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.

Answer 4

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!

Answer 5

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;
    }
}

Answer 6

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()
        }
    }

}

Answer 7

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.

Answer 8

 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

Answer 9

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.

Answer 10

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?

Answer 11

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).

Answer 12

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.

Answer 13

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

Answer 14

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
            }
        }
    }

Answer 15

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.