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

feat: translate embedded text within FMS SearchInput #4773

Merged
merged 4 commits into from
Jun 18, 2024

Conversation

HeartSquared
Copy link
Contributor

@HeartSquared HeartSquared commented Jun 18, 2024

Why

KZN-2449

Support request. Customer noticed some strings in FilterMultiSelect were not translated.

What

Translate embedded text in:

  • FilterMultiSelect.SearchInput
  • InputSearch

@HeartSquared HeartSquared requested a review from a team as a code owner June 18, 2024 03:04
Copy link

changeset-bot bot commented Jun 18, 2024

🦋 Changeset detected

Latest commit: 947d770

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@kaizen/components Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

const placeholder = formatMessage({
id: "filterMultiSelectSearchInput.placeholder",
defaultMessage: "Search…",
description: "Placeholder for the search input",
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think this will be enough for the translation context? I always feel like with less content that's being translated we should provide more context for it - then again, in the grand scheme of this is a placeholder text and it is a generic prompt, sooo maybe give it a crack and see if they respond needing more context :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No idea tbh. Happy to change it if you can think of something more detailed to put. I couldn't think of how else to describe it though 😂

Copy link
Contributor

Choose a reason for hiding this comment

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

Little verbose but maybe something like "Placeholder text for a search input to prompt users to enter a search term"? Not essential maybe lets add it and see if any issues arise

Copy link
Contributor

✨ Here is your branch preview! ✨

Last updated for commit 947d770: chore: update locales

@HeartSquared HeartSquared merged commit d64538c into main Jun 18, 2024
17 checks passed
@HeartSquared HeartSquared deleted the KZN-2449/translate-fms branch June 18, 2024 06:24
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.

2 participants