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

Avoid auto initialising initAll and export Components #759

Merged
merged 2 commits into from
Jun 4, 2018

Conversation

NickColley
Copy link
Contributor

Ensures the user is in control over when the JavaScript executes.

Allows users to execute the components individually, from the
browser bundle.

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-759 June 4, 2018 12:01 Inactive
@kr8n3r
Copy link

kr8n3r commented Jun 4, 2018

does this require a change in our docs?

@NickColley NickColley force-pushed the avoid-auto-initialising-components branch from fc68175 to 657c703 Compare June 4, 2018 13:22
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-759 June 4, 2018 13:23 Inactive
@NickColley
Copy link
Contributor Author

@igloosi sorry, I got sidetracked, this now includes docs and changelog updates.

Ensures the user is in control over when the JavaScript executes.

Allows users to execute the components individually, from the
browser bundle.
@NickColley
Copy link
Contributor Author

Since documenting this way to include JavaScript is a bigger task I've raised an issue: #757

This is since I want to add docs on how to bring the build in from npm, similar to how we're doing it on the Prototype Kit.

Copy link
Contributor

@alex-ju alex-ju left a comment

Choose a reason for hiding this comment

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

💯

@NickColley NickColley merged commit 1b73dcd into master Jun 4, 2018
@NickColley NickColley deleted the avoid-auto-initialising-components branch June 4, 2018 13:42
'Radios'
])
})
it('exported Components can be initialised', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Great test! 💯

@hannalaakso hannalaakso mentioned this pull request Jun 14, 2018
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.

4 participants