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

Tidy up untranslated content and formatting on accordion docs #1958

Merged
merged 1 commit into from
Apr 12, 2021

Conversation

owenatgov
Copy link
Contributor

@owenatgov owenatgov commented Mar 4, 2021

What

User facing: Moves some loose content into our translation file.

Docs facing: Tidies up some of the accordion documentation, specifically removing unnecessary quote marks on single line content.

Why

Principally best practice changes. Serving our content via translation files allows us to easily serve alternative content by locale in the event that it's required.

Addresses #1941 and @andysellick's comment on #1946

No visual changes.

@bevanloon bevanloon temporarily deployed to govuk-publis-tidy-up-tr-cg7n7s March 4, 2021 14:51 Inactive
@owenatgov owenatgov force-pushed the tidy-up-translations-and-docs branch from 50204e9 to 7e82b9c Compare March 4, 2021 14:52
@bevanloon bevanloon temporarily deployed to govuk-publis-tidy-up-tr-cg7n7s March 4, 2021 14:53 Inactive
@owenatgov owenatgov marked this pull request as ready for review March 4, 2021 14:57
@owenatgov owenatgov force-pushed the tidy-up-translations-and-docs branch 2 times, most recently from 750aa4a to 61e6907 Compare March 16, 2021 16:09
@owenatgov owenatgov force-pushed the tidy-up-translations-and-docs branch 4 times, most recently from bec3a08 to 3a30a7c Compare April 1, 2021 10:41
@owenatgov owenatgov force-pushed the tidy-up-translations-and-docs branch 3 times, most recently from e23a2e5 to 656c716 Compare April 7, 2021 16:33
@owenatgov owenatgov requested a review from injms April 7, 2021 16:43
Copy link
Contributor

@injms injms left a comment

Choose a reason for hiding this comment

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

Super 👍

@owenatgov owenatgov force-pushed the tidy-up-translations-and-docs branch from 656c716 to f66551a Compare April 12, 2021 10:48
CHANGELOG.md Outdated Show resolved Hide resolved
@owenatgov owenatgov force-pushed the tidy-up-translations-and-docs branch from f66551a to b606680 Compare April 12, 2021 10:54
@owenatgov owenatgov merged commit 7046ceb into master Apr 12, 2021
@owenatgov owenatgov deleted the tidy-up-translations-and-docs branch April 12, 2021 11:00
alex-ju added a commit that referenced this pull request Apr 15, 2021
* Tidy up untranslated content and formatting on accordion docs ([PR #1958](#1958)) PATCH
* Add visual regression testing tool Percy ([PR #1013](#1013)) PATCH
* Remove `@extend` from component Sass ([PR #2002](#2002)) PATCH
* Remove grid overrides in layout-header ([PR #2013](#2013)) PATCH
@alex-ju alex-ju mentioned this pull request Apr 15, 2021
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.

4 participants