Skip to content

Cell disappear in 0.1.1 when selected - #4

Closed
siutsin wants to merge 1 commit into
CSStickyHeaderFlowLayout:masterfrom
siutsin:develop
Closed

Cell disappear in 0.1.1 when selected#4
siutsin wants to merge 1 commit into
CSStickyHeaderFlowLayout:masterfrom
siutsin:develop

Conversation

@siutsin

@siutsin siutsin commented Apr 2, 2014

Copy link
Copy Markdown

0.1.1:
0 1 1

0.1.0:
0 1 0

It seems OK after removing attributes.zIndex = 1;

jamztang pushed a commit that referenced this pull request Apr 5, 2014
@jamztang

jamztang commented Apr 5, 2014

Copy link
Copy Markdown
Collaborator

Hi @siutsin, thanks for the bug reporting. It's originally intended to guarantee the sticky header is not overlapping the cells.

According to my testing at a testing branch at issue/#4, the bug doesn't happened. Your comment also state you're using dequeueReusableCellAtIndexPath: in didSelectItemAtIndexPath:, please try -[UICollectionView cellForRowAtIndexPath:] instead .

If that doesn't solve your problem, please create a branch that demonstrate the problem, so we're on the same page :)

@siutsin

siutsin commented Apr 6, 2014

Copy link
Copy Markdown
Author

Thanks James! It works for cellForItemAtIndexPath but dequeueReusableCellAtIndexPath.
A bit test on the cell after fix it: NSLog(@"cell reuse id: %@", cell.reuseIdentifier); and the reuseIdentifier is correct. Weird.

@jamztang

jamztang commented Apr 6, 2014

Copy link
Copy Markdown
Collaborator

You shouldn't be using dequeueReusableCellAtIndexPath Anyway. What's weird if the reuse identifier is correct?

@siutsin

siutsin commented Apr 6, 2014

Copy link
Copy Markdown
Author

I thought the disappearing of the cell is due to retrieve the wrong identifier cell before and obviously it is not.
Thanks for you comment and I guess I used the wrong approach :) 👍

@jamztang

jamztang commented Apr 6, 2014

Copy link
Copy Markdown
Collaborator

No problem :)

@jamztang jamztang closed this Apr 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants