Skip to content
This repository was archived by the owner on Sep 8, 2020. It is now read-only.

dynamically add calendar object to parent scope #10 #12

Closed
wants to merge 1 commit into from

Conversation

joshkurz
Copy link
Contributor

Adds a dynamic calendar object to the parent scope. This allows the user to create multiple calendars on the same scope with different names and not have them clash with each other.

The calendar is still using a child scope so that the functions can be passed in correctly.

calendar = elm.html('');
}
else{
calendar = angular.element(elm.parent()).scope()[attrs.calendar] = elm.html('');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

elm.parent() already returns jqlite/jquery element that has .scope() on it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gotcha... I think we should just close this PR because #13 is more robust and gives the calendar more isolation without rendering functionality which is always good.

@yourilima yourilima mentioned this pull request Apr 11, 2013
@joshkurz joshkurz closed this Apr 12, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants