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

This fixes deprecation warnings for iOS 8-based targets #4

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

Conversation

emaloney
Copy link

When building for a target with a minimum platform of iOS 8, additional deprecation warnings will occur.

This pull request fixes those warnings.

• CLCalendarPastDayNumberTextColor — The day-of-month number text color for dates in the past
• CLCalendarFutureDayNumberTextColor – The day-of-month number text color for dates in the future
• CLCalendarCurrentDayNumberTextColor — The day-of-month number text color for today
• CLCalendarSelectedDayNumberTextColor — The day-of-month number text color for the selected day when that date is not today
• CLCalendarSelectedCurrentDayNumberTextColor — The day-of-month number text color for the selected day when that date is today
• CLCalendarCurrentDayNumberBackgroundColor — The background color to use for the highlight circle behind the current day when that date isn't selected
• CLCalendarSelectedDayNumberBackgroundColor — The background color to use for the highlight circle behind the selected day when that date is not today
• CLCalendarSelectedCurrentDayNumberBackgroundColor — The background color to use for the highlight circle behind the selected day when that date is today
@emaloney emaloney closed this Mar 23, 2015
@emaloney
Copy link
Author

Will redo this. I saw that a commit intended for a separate pull request made it into here.

@emaloney emaloney reopened this Mar 23, 2015
@emaloney emaloney closed this Mar 23, 2015
@emaloney emaloney reopened this Mar 23, 2015
@emaloney
Copy link
Author

Fixed.

jmschonfeld pushed a commit to Jeretech/CLWeeklyCalendarView that referenced this pull request Jul 4, 2015
@esusatyo
Copy link

esusatyo commented Feb 2, 2016

Is this going to be merged to master branch?

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.

None yet

3 participants