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

[core] Add a note for the Y.config.core documentation #1682

Open
wants to merge 1 commit into
base: live-docs
Choose a base branch
from

Conversation

andrewnicols
Copy link
Contributor

No description provided.

@caridy
Copy link
Member

caridy commented Mar 7, 2014

I'm not so sure that msg reflects the reality about core. In fact, I don't recommend people replacing that thru YUI_config directly, because we might change what is in that list and it will break their app. Instead, we recommend to do this:

YUI.Env.core.push.apply(YUI.Env.core, YUI_config.extendedCore || []);
YUI(); // first instance in the page...

that does the trick, adding new stuff into core from a new config variable called extendedCore (which is not YUI specific), and does not break or mess with the original core definition.

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