-
Notifications
You must be signed in to change notification settings - Fork 236
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gemmaleigh
force-pushed
the
shim-links-with-aria-role-button
branch
from
August 19, 2016 12:18
8ea8607
to
6d3efeb
Compare
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
force-pushed
the
shim-links-with-aria-role-button
branch
from
August 19, 2016 12:19
6d3efeb
to
baf595f
Compare
👍 |
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
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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