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

Update default Exit This Page button text #3989

Merged
merged 2 commits into from
Jul 28, 2023
Merged

Conversation

querkmachine
Copy link
Member

@querkmachine querkmachine commented Jul 20, 2023

Updates the default Exit This Page button text to include visually hidden text, so the button reads 'Emergency Exit this Page' to screen reader users, to clarify the button's intended use as a safety feature.

Split out from #3878. Intended to resolve #3688.

I assume this would need a 'Suggested change' in the changelog now that EtP has shipped in 4.7.0?

@querkmachine querkmachine requested a review from a team July 20, 2023 14:46
@querkmachine querkmachine self-assigned this Jul 20, 2023
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3989 July 20, 2023 14:47 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3989 July 27, 2023 15:57 Inactive
@colinrotherham colinrotherham self-requested a review July 27, 2023 16:06
Copy link
Contributor

@colinrotherham colinrotherham left a comment

Choose a reason for hiding this comment

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

Code looks good, happy to approve

Probably do need that CHANGELOG entry

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3989 July 27, 2023 17:25 Inactive

The default text of the Exit this Page button has been updated to indicate that the button is a safety tool and not a generic method of leaving the current page. If you are using the component's default label, you may wish to update it to the new value.

```diff
Copy link
Contributor

Choose a reason for hiding this comment

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

😮


```diff
<a href="..." role="button" draggable="false" class="govuk-button govuk-button--warning govuk-exit-this-page__button govuk-js-exit-this-page-button" data-module="govuk-button">
- Exit this page
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

```diff
<a href="..." role="button" draggable="false" class="govuk-button govuk-button--warning govuk-exit-this-page__button govuk-js-exit-this-page-button" data-module="govuk-button">
- Exit this page
+ <span class="govuk-visually-hidden">Emergency</span> Exit this page
Copy link
Contributor

Choose a reason for hiding this comment

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

🙌

@querkmachine querkmachine merged commit 7393b5b into main Jul 28, 2023
39 checks passed
@querkmachine querkmachine deleted the update-etp-default-text branch July 28, 2023 08:28
romaricpascal pushed a commit that referenced this pull request Aug 7, 2023
Update default Exit This Page button text
@romaricpascal romaricpascal mentioned this pull request Dec 8, 2023
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.

The non-visual experience of Exit this Page button doesn't make it clear it's a safety feature
3 participants