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

Added bottom arrow positions, single date selection and auto select date features #15

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nayeb
Copy link

@nayeb nayeb commented Jul 19, 2012

No description provided.

nayeb added 3 commits July 19, 2012 18:26
…top/bottom in addition to the horizontal arrow positions left/center/right
Added autoSelectDate feature:
After range select or date select in single date mode, the calendar
view closes itself and populates the chosen date or date range.

Single date range bug fixes:
I did broke the range selection previously. Now its fixed again.
And now the endDate will also be set to the startDate if you set the
startDate property on the calendar view controller
@ocrickard
Copy link
Owner

Thanks for the help! I'm going to merge these three pull requests tonight. Sorry for the delay. I've been moving and starting a new job!

@ocrickard
Copy link
Owner

Just tested your code, and it looks like there's a couple things missing. Won't compile on my computer with a fresh pull. First it wouldn't recognize the single selection enum because you had not imported the OCCalendarView.h file. Second, it appears in the demo project that your are using a different implementation of the init statement that is not included in the OCCalendar source files. I believe that this may be just because a couple of files were not added to the commit. Let me know if these issues get fixed. Of course I apologize in advance if I have mis-diagnosed the problem.

…. Added missing init method for the ViewController.
@nayeb
Copy link
Author

nayeb commented Jul 24, 2012

Thanks for the hints. I fixed everything you mentioned and also implemented the ability to change the selectionColor and the todayMarkerColor. Without changing these settings, everything should behave as it did bevor (defaults to those red colors you had chosen).

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