modus-web-components icon indicating copy to clipboard operation
modus-web-components copied to clipboard

[Modus-Table] Allow translated strings for pagination and sorting.

Open janwiebe-glasbergen opened this issue 1 year ago • 3 comments

Prerequisites

Proposal

The modus-table component contains some hardcoded English strings. As shown in the below screenshot: image

As a developer I would like to be able to provide custom values for these strings, so that I can support multiple languages.

Motivation and context

We are planning to release a UI component, that uses the modus-table, which will be used in a multi-language product.

What is the issue regarding ?

@trimble-oss/modus-web-components

Priority

High

What product/project are you using Modus Components for ?

Nova / Construction calculation platform UI

What is your team/division name ?

ACDC

Are you willing to contribute ?

Maybe

Are you using Modus Web Components in production ?

No response

janwiebe-glasbergen avatar Feb 02 '24 10:02 janwiebe-glasbergen

Hello @janwiebe-glasbergen! Thanks for opening an issue. The Modus core team will get back to you soon (usually within 24-hours) and provide guidance on how to proceed. Contributors are welcome to participate in the discussion and provide their input on how to best solve the issue, and even submit a PR if they want to.

Please wait until the issue is ready to be worked on before submitting a PR, or you can reach out to the core team if it is time bound. For trivial things, or bugs that don't change the expected behaviors and UI, you can go ahead and make a PR.

github-actions[bot] avatar Feb 02 '24 10:02 github-actions[bot]

@prashanth-offcl to check and decide on the approach for translation.

prashanth-offcl avatar Aug 14 '24 14:08 prashanth-offcl

@prashanth-offcl Any updates on this issue?

janwiebe-glasbergen avatar Sep 30 '24 05:09 janwiebe-glasbergen

Hi @prashanth-offcl attached the analysis documentation for this ticket. Kindly refer https://docs.google.com/document/d/1qtB8AmoEUqhIJEOQ3yXJ3MLfOPa4di-qkPZ3j9MvdIQ/edit?tab=t.0

prashanthr6383 avatar Oct 24 '24 08:10 prashanthr6383

We think we should support French, Finnish, Spanish, German, and Italian to begin with. But would like to start with the requester's language. It appears to be French. Could you @janwiebe-glasbergen provide appropriate translations and confirm the language?

enowak1031 avatar Nov 18 '24 15:11 enowak1031

Prashanth talked to the requester. He needs French and German and later Dutch.

enowak1031 avatar Nov 19 '24 15:11 enowak1031