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

Add new example page of all translated components to review app #2809

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

vanitabarrett
Copy link
Contributor

@vanitabarrett vanitabarrett commented Aug 24, 2022

Note: this relies on @querkmachine 's change to allow translation of the character count fallback message. Please only review the second commit.

Closes #2692

What

Adds a new example page containing all GOV.UK Frontend components, translated into Welsh. This includes translation of text in Nunjucks/HTML, not strings in component JavaScript code as we haven't begun our implementation to allow that yet.

This should also include translation of any HTML attributes, e.g: aria labels.

Note: the card was originally written with the end goal of having a full page realistic example of translated components, but it didn't feel possible to have a realistic page with all components in, so I've instead opted for just a "list" of all components on a page instead.

Why

  • So we can view examples of all translated components in one place.
  • We can spot components which may be missing abilities to translate.
  • If we're looking to make design changes in future and we want to check how it affects components in a different language.

➡️ Preview link: https://govuk-frontend-pr-2809.herokuapp.com/examples/translated

@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2809 August 24, 2022 16:07 Inactive
@vanitabarrett vanitabarrett changed the base branch from main to kg-i18n-character-count August 24, 2022 16:08
@vanitabarrett vanitabarrett changed the base branch from kg-i18n-character-count to main August 24, 2022 16:09
@vanitabarrett vanitabarrett force-pushed the vbs-add-i18n-component-example-page branch from 4d5d5be to ba6cbc8 Compare August 24, 2022 16:12
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2809 August 24, 2022 16:12 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2809 August 26, 2022 13:20 Inactive
@vanitabarrett vanitabarrett force-pushed the vbs-add-i18n-component-example-page branch from ba6cbc8 to bd249ff Compare August 30, 2022 10:22
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2809 August 30, 2022 10:22 Inactive
Have translated all strings that are currently possible to translate
(should be everything apart from strings in hardcoded JS)
@vanitabarrett vanitabarrett force-pushed the vbs-add-i18n-component-example-page branch from bd249ff to 345e8cd Compare August 30, 2022 14:19
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-2809 August 30, 2022 14:19 Inactive
@vanitabarrett vanitabarrett marked this pull request as ready for review August 30, 2022 14:24
Copy link
Member

@querkmachine querkmachine left a comment

Choose a reason for hiding this comment

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

Looks good here 👍

@vanitabarrett vanitabarrett merged commit dc55e43 into main Aug 30, 2022
@vanitabarrett vanitabarrett deleted the vbs-add-i18n-component-example-page branch August 30, 2022 14:47
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.

Add full page example(s) for translated components (via macro options) in the review app
4 participants