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

Include the shim-links-with-button-role.js script from the govuk_frontend_toolkit #237

Merged
merged 3 commits into from
Aug 22, 2016

Conversation

gemmaleigh
Copy link
Contributor

@gemmaleigh gemmaleigh commented Aug 11, 2016

Include a script from the govuk frontend toolkit to shim anchors with
role="button" to be 'clicked' via the space key.

For context, here is the PR adding this script to the govuk_frontend_toolkit.
alphagov/govuk_frontend_toolkit#297

@gemmaleigh gemmaleigh force-pushed the shim-links-with-aria-role-button branch from 8ea8607 to 6d3efeb Compare August 19, 2016 12:18
@gemmaleigh gemmaleigh changed the title Include the anchor buttons script from the frontend toolkit Include the shim-links-with-button-role.js script from the govuk_frontend_toolkit Aug 19, 2016
Links styled to look like buttons lack button behaviour. This script will allow them to be triggered with a space key after they’ve been focused, to match standard buttons.

By default, this behaviour will only be applied to links with a role of button.
By default, this behaviour will only be applied to links with a role of button.
<a class="button" role="button">A button</a>

This script will allow them to be triggered with a space key after they’ve been focused, to match standard buttons.
@gemmaleigh gemmaleigh force-pushed the shim-links-with-aria-role-button branch from 6d3efeb to baf595f Compare August 19, 2016 12:19
@tvararu
Copy link
Contributor

tvararu commented Aug 22, 2016

👍

@tvararu tvararu merged commit efce8ec into master Aug 22, 2016
@tvararu tvararu deleted the shim-links-with-aria-role-button branch August 22, 2016 09:08
gemmaleigh added a commit that referenced this pull request Oct 7, 2016
Logo fixes: Fix logo breaking layout when increasing font size, fix
invalid or non-standard CSS and remove obsolete CSS around logo, adjust
logo dimensions slightly (PR #237)
robinwhittleton pushed a commit that referenced this pull request Oct 13, 2016
Breaking changes:

- #244 Migrate documentation into a seperate application

All changes:

- Bump all GOV.UK assets to their latest versions
- Remove duplicate GOV.UK assets copied to the app
- #241 Warn against using the prototype kit to build production services
- #268 Automatically keep the latest release branch up to date. This can be used to update the kit
- #270 Add a new stylesheet for the unbranded layout to fix font issues
- #257 Make CSS output easier to debug (with sourcemaps)
- #237 Make links with role="button" behave like buttons
- #224 Lint the prototype kit’s codebase using Standard. This only applies to the kit’s codebase - there’s no requirement for your app to meet this
robinwhittleton pushed a commit that referenced this pull request Oct 13, 2016
Breaking changes:

- #244 Migrate documentation into a separate application

All changes:

- Bump all GOV.UK assets to their latest versions
- Remove duplicate GOV.UK assets copied to the app
- #241 Warn against using the prototype kit to build production services
- #268 Automatically keep the latest release branch up to date. This can be used to update the kit
- #270 Add a new stylesheet for the unbranded layout to fix font issues
- #257 Make CSS output easier to debug (with sourcemaps)
- #237 Make links with role="button" behave like buttons
- #224 Lint the prototype kit’s codebase using Standard. This only applies to the kit’s codebase - there’s no requirement for your app to meet this
@robinwhittleton robinwhittleton mentioned this pull request Oct 13, 2016
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