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 Warning component on accessibility audits page #191

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Conversation

adyhoran1
Copy link
Collaborator

Typo fixed, content tweaked so it's easier to read.

Typo fixed, content tweaked so it's easier to read.

Signed-off-by: adyhoran1 <[email protected]>
@@ -11,8 +11,8 @@
<div class="govuk-warning-text">
<span class="govuk-warning-text__icon" aria-hidden="true">!</span>
<strong class="govuk-warning-text__text">
<span class="govuk-warning-text__assistive">Warning</span>
Services built using the prototype kit, must not be presented for audits. They are not intended for production quality products or services.
<span class="govuk-warning-text__assistive"> Warning </span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you explain why you've made these changes to the warning component and put the text in quotes.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@andyjones81 the warning component is currently not displaying correctly on the page:

image

This was my attempt at fixing it 🙂

@andyjones81 andyjones81 self-assigned this May 5, 2024
@andyjones81 andyjones81 self-requested a review May 5, 2024 08:12
@adyhoran1 adyhoran1 changed the title Update index.html Update Warning component index.html Jun 11, 2024
@adyhoran1 adyhoran1 changed the title Update Warning component index.html Update Warning component on accessibility audits page Jun 11, 2024
@andyjones81 andyjones81 merged commit 9e04f99 into main Nov 4, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants