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.

Sign up for GitHub

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

Jump to bottom

Calendar Item isSelectable() method #14

Open
bkach opened this issue Dec 5, 2017 · 2 comments
Open

Calendar Item isSelectable() method #14

bkach opened this issue Dec 5, 2017 · 2 comments

Comments

@bkach
Copy link

bkach commented Dec 5, 2017

In the CalendarItem there is an isSelectable() method. It currently does not tell you whether the item is selectable, and instead tells you if it IS selected.

It should be renamed to isSelected()
@bkach
Copy link
Author

bkach commented Dec 5, 2017 •
edited
Loading

Or is it the other way around? Does the variable selected need to be changed to selectable?

Edit: This seems to be the case. The variable selected should be changed to selectable within the CalendarItem

@francesco-furlan
Copy link
Contributor

@bkach yep, you're right the variable selected should be changed to selectable.
I'm going to do this refactor ASAP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bkach @francesco-furlan