Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cellState of cell in didSelectDate not updated #558

Closed
jokerham opened this issue Sep 10, 2017 · 7 comments
Closed

cellState of cell in didSelectDate not updated #558

jokerham opened this issue Sep 10, 2017 · 7 comments
Assignees

Comments

@jokerham
Copy link

jokerham commented Sep 10, 2017

Version is 7.0.6
While debugging didSelectDate function, I have found that the cellState of the cell in the parameter and cellState in the parameter does not match.

2017-09-10 9 40 50

as you can see the cellState variable shows that isSelected = true, while calendarCell.cellState.isSelected = false.
I am trying to achieve some layout functions in setTheme based on calendarCell variable, but since the status is not update, it does not show correctly.

Do I have to change the state myself?

@patchthecode
Copy link
Owner

patchthecode commented Sep 10, 2017

you have to use cellState.
Here is a discussion on this -> #553

I am thinking of removing isSelected on the cell.
CellState problem is shown on Problem#2

If you have some good ideas that i can use to not remove the isSelected feature on a cell, then let me know. And i can incorporate them into the next release.

@patchthecode
Copy link
Owner

closing issue. Let me know if you still need more information on this.

@jokerham
Copy link
Author

I was not referring to the isSelected of the cell, but the cellState. Just that the cellState on the cell(p) of parameter and the cellState(P) on the parameter differs. I can always resolve it by overriding the cellState of the cell(p) from cellState(P) received.

@patchthecode
Copy link
Owner

I do not understand.
✅ We only have on cellState which is given by the function.
❎ We do not have a cellState on the cell.

Can you explain some more by joining me here? -> https://gitter.im/patchthecode/JTAppleCalendar

@patchthecode
Copy link
Owner

Closing again until response.

@jokerham
Copy link
Author

My mistake... you can close...

@patchthecode
Copy link
Owner

Ah thanks for responding.
If you have any other issues, then let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants