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

Allow aria-controls attribute on search component #1203

Merged
merged 2 commits into from
Nov 20, 2019

Conversation

kr8n3r
Copy link
Contributor

@kr8n3r kr8n3r commented Nov 19, 2019

What

This is adding the ability to specify the aria-controls attribute, that's available on the input component.

Why

On finders we are looking to upgrade the text input with a search icon to a search component.

The latter is currently missing this attribute which is being used to announce dynamic changes.

View Changes

https://govuk-publishing-compo-pr-1203.herokuapp.com/component-guide/search

@kr8n3r kr8n3r changed the title Allow aria-control attribute on search component Allow aria-controls attribute on search component Nov 19, 2019
@bevanloon bevanloon temporarily deployed to govuk-publishing-compo-pr-1203 November 19, 2019 12:43 Inactive
@kr8n3r kr8n3r force-pushed the allow-aria-attributes-on-search-component branch from 8c87ea1 to 49ae5bc Compare November 19, 2019 12:44
@bevanloon bevanloon temporarily deployed to govuk-publishing-compo-pr-1203 November 19, 2019 12:44 Inactive
@kr8n3r kr8n3r force-pushed the allow-aria-attributes-on-search-component branch from 49ae5bc to 474a3e9 Compare November 19, 2019 12:48
@bevanloon bevanloon temporarily deployed to govuk-publishing-compo-pr-1203 November 19, 2019 12:48 Inactive
This is adding the ability to specify the aria-controls attribute,
that's available on the input component.

Why
On finders we are looking to upgrade the text input with a search icon
to a search component.

The latter is currently missing this attribute which is being used
to announce dynamic changes.
@kr8n3r kr8n3r merged commit a6fd2b1 into master Nov 20, 2019
@kr8n3r kr8n3r deleted the allow-aria-attributes-on-search-component branch November 20, 2019 14:06
kr8n3r added a commit that referenced this pull request Nov 20, 2019
Contains

* Update govuk-frontend to 3.4.0 ([PR #1204](#1204))

* Allow aria-controls attribute on search component ([PR #1203](#1203))
@kr8n3r kr8n3r mentioned this pull request Nov 20, 2019
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.

3 participants