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

Add accordion component #635

Merged
merged 2 commits into from
Jan 10, 2019
Merged

Add accordion component #635

merged 2 commits into from
Jan 10, 2019

Conversation

hannalaakso
Copy link
Member

@hannalaakso hannalaakso commented Nov 14, 2018

Note for reviewers:

Please leave any comments pertaining to the guidance/documentation or the content used for the accordion on this PR.

Please leave any comments for the accordion component itself (design, code) on alphagov/govuk-frontend#958.

@hannalaakso hannalaakso changed the title Add accordion as a pre-release DNM: Add accordion as a pre-release Nov 14, 2018
@govuk-design-system-ci
Copy link
Collaborator

govuk-design-system-ci commented Nov 14, 2018

You can preview this change here:

Built with commit 402e483

https://deploy-preview-635--govuk-design-system-preview.netlify.com

@NickColley
Copy link
Contributor

We can consider using the 'set capture' https://mozilla.github.io/nunjucks/templating.html#set like we do with the tabs component.

@aliuk2012 aliuk2012 force-pushed the accordion-prerelease branch from 652a798 to 54ef088 Compare December 17, 2018 17:02
@aliuk2012 aliuk2012 force-pushed the accordion-prerelease branch 2 times, most recently from bd4d932 to 1c89f2a Compare January 10, 2019 16:04
@aliuk2012 aliuk2012 changed the title DNM: Add accordion as a pre-release Add accordion component Jan 10, 2019
@aliuk2012 aliuk2012 force-pushed the accordion-prerelease branch from 1c89f2a to 508eb8a Compare January 10, 2019 16:12
@aliuk2012 aliuk2012 force-pushed the accordion-prerelease branch from 508eb8a to c33a651 Compare January 10, 2019 16:19
Correction in research section to clarify it was the design not the component that was originally tested by GDS.
Copy link
Contributor

@amyhupe amyhupe left a comment

Choose a reason for hiding this comment

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

All looks good now 👍

@aliuk2012 aliuk2012 merged commit 65c2a3e into master Jan 10, 2019
@NickColley NickColley deleted the accordion-prerelease branch January 10, 2019 17:34
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.

5 participants