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(atomic): localize tab name & label #5071

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

louis-bompart
Copy link
Collaborator

@louis-bompart louis-bompart commented Mar 13, 2025

Adapt the atomic-tab-manager.tsx component to use the i18n library for localization of the label and name parameters.

  • Update the label and name properties to use this.bindings.i18n.t for localization.
  • Add a new test file atomic-tab-manager.spec.tsx to check if the label and name parameters are localized using bindings.i18n.t.

For more details, open the Copilot Workspace session.

@louis-bompart louis-bompart requested a review from a team as a code owner March 13, 2025 20:58
@louis-bompart louis-bompart marked this pull request as draft March 13, 2025 20:59
Copy link

Pull Request Report

PR Title

✅ Title follows the conventional commit spec.

Live demo links

Bundle Size

File Old (kb) New (kb) Change (%)
case-assist 246.6 246.6 0
commerce 359.2 359.2 0
search 417.3 417.3 0
insight 408.5 408.5 0
recommendation 258 258 0
ssr 411.1 411.1 0
ssr-commerce 375.7 375.7 0

Verified

This commit was signed with the committer’s verified signature.
louis-bompart Louis Bompart
Adapt the `atomic-tab-manager.tsx` component to use the i18n library for localization of the `label` and `name` parameters.

* Update the `label` and `name` properties to use `this.bindings.i18n.t` for localization.
* Add a new test file `atomic-tab-manager.spec.tsx` to check if the `label` and `name` parameters are localized using `bindings.i18n.t`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/coveo/ui-kit?shareId=XXXX-XXXX-XXXX-XXXX).
@louis-bompart louis-bompart force-pushed the louis-bompart/localize-tab-label branch from 1700193 to 43c788b Compare March 13, 2025 21:53
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.

None yet

1 participant