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

Fixed: CPDatePicker does not grey out dates outside of minDate and maxDate #2297

Merged
merged 1 commit into from Jan 15, 2015

Conversation

daboe01
Copy link
Contributor

@daboe01 daboe01 commented Jan 15, 2015

Previously, CPDatePicker did not disable the dates outside of minDate and maxDate. This left the user wondering why some dates are not clickable for no apparent reason.
This PR sets the state to disabled if the tile not within minDate and maxDate.

Fixes #2262

@cappbot cappbot added this to the Someday milestone Jan 15, 2015
@cappbot cappbot added the #new label Jan 15, 2015
@ahankinson
Copy link
Contributor

Wow! Nice one. It's great when such a nice feature takes so little code. :)

-#new
+feature
+AppKit
+#needs-review
milestone=0.9.8

@cappbot
Copy link

cappbot commented Jan 15, 2015

Milestone: Someday. Label: #new. What's next? A reviewer should examine this issue.

@cappbot
Copy link

cappbot commented Jan 15, 2015

Milestone: 0.9.8. Labels: #needs-review, AppKit, feature. What's next? This issue is pending an architectural or implementation design decision and should be discussed or voted on.

primalmotion added a commit that referenced this pull request Jan 15, 2015
Fixed: CPDatePicker does not grey out dates outside of minDate and maxDate
@primalmotion primalmotion merged commit a2c0b47 into cappuccino:master Jan 15, 2015
@primalmotion
Copy link
Member

+#fixed

Thanks

@cappbot
Copy link

cappbot commented Jan 15, 2015

Milestone: 0.9.8. Labels: #fixed, AppKit, feature. What's next? This issue is considered successfully resolved.

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

Successfully merging this pull request may close these issues.

CPDatePicker does not grey out dates outside of minDate and maxDate [+1]
4 participants