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

DNM: Add News and Comms with sorting and small form controls #1212

Closed
wants to merge 18 commits into from

Conversation

hannalaakso
Copy link
Member

This adds the News and Comms page for the accessibility audit.

It's branched off #1125 to get the new small form controls work.

We've removed the other full page examples from this branch as we intend to give the version in this branch to test separately.

Dave House and others added 18 commits February 20, 2019 17:12
This will allow us to shuffle a data set in a deterministic way

Co-authored-by: Nick Colley <nick.colley@digital.cabinet-office.gov.uk>
Since we're shuffling one data set we don't need the different
variations anymore

Co-authored-by: Nick Colley <nick.colley@digital.cabinet-office.gov.uk>
Add small form inputs to view.

Remove error validation from filters as we cover this with
another example.

Co-authored-by: Nick Colley <nick.colley@digital.cabinet-office.gov.uk>
We only want to to test News and Comms on this branch
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-pr-1212 February 21, 2019 13:22 Inactive
hannalaakso added a commit that referenced this pull request Feb 21, 2019
We will test it in a separate branch:
#1212
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-review-audit February 26, 2019 14:10 Inactive
@hannalaakso hannalaakso changed the title Add News and Comms with sorting and small form controls DNM: Add News and Comms with sorting and small form controls Feb 26, 2019
@NickColley
Copy link
Contributor

We have created a permanent staging application for this branch so that heroku does not consider it stale.

http://govuk-frontend-review-audit.herokuapp.com/full-page-examples/news-and-communications

It should be removed when we are done with this branch.

@36degrees
Copy link
Contributor

Now that small controls has shipped, should we tidy this up and merge it?

@NickColley
Copy link
Contributor

Closing in favour of #1300

@NickColley NickColley closed this Apr 29, 2019
@NickColley NickColley deleted the small-form-controls-for-full-page-template branch April 29, 2019 14:38
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