Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Headers Swedish old translations fixed #2136

Merged
merged 4 commits into from
Apr 16, 2024

Conversation

adoroshk
Copy link
Contributor

@adoroshk adoroshk commented Apr 15, 2024

Summary

This PR replaces terra-notification-dialog headers Swedish translations for locales sv and sv-SE to match the new translations used elsewhere.

Testing

This change was tested using:

  • WDIO
  • Jest
  • Visual testing (please attach a screenshot or recording)
  • Other (please describe below)
  • No tests are needed

New headers for for locales sv and sv-SE locales.

Screenshot 2024-04-15 at 9 43 41 AM Screenshot 2024-04-15 at 9 43 50 AM Screenshot 2024-04-15 at 9 44 00 AM Screenshot 2024-04-15 at 9 44 08 AM

Reviews

In addition to engineering reviews, this PR needs:

  • UX review
  • Accessibility review
  • Functional review

Additional Details

This PR resolves:
UXPLATFORM-10362

Comment on lines 2 to 5
"Terra.notification.dialog.hazard-low": "Information",
"Terra.notification.dialog.error": "Fel",
"Terra.notification.dialog.hazard-medium": "Obs!",
"Terra.notification.dialog.hazard-high": "Varning"
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems like extra white space was added to the strings.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, removed extra spacing in 3b3f5cb

"Terra.notification.dialog.error": "Fel",
"Terra.notification.dialog.hazard-medium": "Varning",
"Terra.notification.dialog.hazard-high": "Avisering"
"Terra.notification.dialog.hazard-low": "Information",
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this the proper translated word? It appears to be English.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was provided that way:
Screenshot 2024-04-15 at 10 22 58 AM

And it looks like it's the same in English and Swedish:
Screenshot 2024-04-15 at 10 25 11 AM

@github-actions github-actions bot temporarily deployed to preview-pr-2136 April 15, 2024 14:23 Destroyed
@github-actions github-actions bot temporarily deployed to preview-pr-2136 April 15, 2024 19:24 Destroyed
@github-actions github-actions bot temporarily deployed to preview-pr-2136 April 16, 2024 13:56 Destroyed
@adoroshk adoroshk merged commit b5db4c5 into main Apr 16, 2024
22 checks passed
@adoroshk adoroshk deleted the terra-notification-dialog-update-sv-translations branch April 16, 2024 14:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants