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

allow a function for day style #6

Merged
merged 1 commit into from
Jan 28, 2016
Merged

Conversation

aksonnic
Copy link
Contributor

Lets developers specify a function for a datasource display function to allow custom day styles, without using renderDay, which is called for every day in the calendar.

@Paul-DS
Copy link
Owner

Paul-DS commented Jan 28, 2016

Good idea !

I've just commit a modification of your code (b1733aa).
I separated the functionality into 2 options :

  • I added a style "custom" that uses the custom method for rendering data source.
  • I added an option "customDataSourceRenderer" for passing the render method.

I prefer separate these options to keep strong typing on the widget options.

It's OK for you ?

@aksonnic
Copy link
Contributor Author

Sure, that's fine. I was trying not to bloat the options object, but I don't use typescript, so I wasn't caring too much about strong typing.

@Paul-DS Paul-DS merged commit e554b4b into Paul-DS:master Jan 28, 2016
@Paul-DS
Copy link
Owner

Paul-DS commented Jan 28, 2016

Ok, great !

Thanks for the contribution !

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