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

Day View Event Can't be selected #75

Closed
bocodigital opened this issue Mar 13, 2015 · 6 comments
Closed

Day View Event Can't be selected #75

bocodigital opened this issue Mar 13, 2015 · 6 comments

Comments

@bocodigital
Copy link

Using CalendarView then switching to CKCalendarViewModeDay then click on event in event table view
(void)calendarView:(CKCalendarView *)CalendarView didSelectEvent:(CKCalendarEvent *)event

does not fire, works in CKCalendarViewModeMonth and CKCalendarViewModeWeek

@bocodigital
Copy link
Author

When i add multiple events on the day i can click the lower ones, looks like the Month navigation has a hidden layer covering the first event in the list in day view

@bocodigital
Copy link
Author

in CKCalendarView.m

I changed
if(displayMode == CKCalendarViewModeDay)
{
// Hide the cells entirely and only show the events table
rect = CGRectMake(0, 0, rect.size.width, cellSize.height);
}

to

if(displayMode == CKCalendarViewModeDay)
{
// Hide the cells entirely and only show the events table
rect = CGRectMake(0, 0, rect.size.width, cellSize.height-60);
}

which I will admit is hacky but it fixed the issue.

@MosheBerman
Copy link
Owner

Hey,

Thanks for reporting the issue. Are you using auto layout? Can you please provide screenshots? Also, what lines did you edit in CKCalendarView.m?

Thanks,
Moshe

@bocodigital
Copy link
Author

I am using autolayout line 228. Also I am using Socket.IO-Client-Swift which requires IOS 8.0 or higher and a lot of the NSCalendar Unit properties are deprecated just FYI. Looks like a pretty simple re-tool, just a little tedious.

@MosheBerman
Copy link
Owner

Yeah, I thought I fixed the deprecations already, but I'll get on that soon. The auto layout is a little trickier, since I'm doing my own cells recycling. I'm concerned about recreating constraints and performance.

@MosheBerman
Copy link
Owner

MBCalendarKit 5.0.0 is out with Auto Layout and other fixes that likely cover this. If it's still relevant to you, check it out. If not, would you mind if I close this issue?

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