Skip to content

Commit

Permalink
[Lens] Fixes color mappings special tockens i18n (elastic#168152)
Browse files Browse the repository at this point in the history
## Summary

fix elastic#168151

To test:
- create a cartesian charts with breakdowns that contains both the
`Other` group and an `(empty)` value
- apply a manual color mapping that use both these terms by select them
from the combo-box, not by typing them.
- Then change the Kibana language to a different one other than English
and see if the color mapping applies correctly.

---------

Co-authored-by: Stratoula Kalafateli <[email protected]>
  • Loading branch information
markov00 and stratoula authored Oct 6, 2023
1 parent 866edd2 commit c828ff4
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
* Side Public License, v 1.
*/

import { i18n } from '@kbn/i18n';
import { ColorMapping } from '../config';

export function ruleMatch(
Expand Down Expand Up @@ -41,6 +42,16 @@ export function rangeMatch(rule: ColorMapping.RuleRange, value: number) {

// TODO: move in some data/table related package
export const SPECIAL_TOKENS_STRING_CONVERTION = new Map([
['__other__', 'Other'],
['', '(empty)'],
[
'__other__',
i18n.translate('coloring.colorMapping.terms.otherBucketLabel', {
defaultMessage: 'Other',
}),
],
[
'',
i18n.translate('coloring.colorMapping.terms.emptyLabel', {
defaultMessage: '(empty)',
}),
],
]);
2 changes: 2 additions & 0 deletions x-pack/plugins/translations/translations/fr-FR.json
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,8 @@
"coloring.dynamicColoring.rangeType.label": "Type de valeur",
"coloring.dynamicColoring.rangeType.number": "Numéro",
"coloring.dynamicColoring.rangeType.percent": "Pourcent",
"coloring.colorMapping.terms.otherBucketLabel": "Autre",
"coloring.colorMapping.terms.emptyLabel": "(vide)",
"console.helpPage.learnAboutConsoleAndQueryDslText": "En savoir plus sur {console} et {queryDsl}",
"console.historyPage.itemOfRequestListAriaLabel": "Requête : {historyItem}",
"console.settingsPage.refreshInterval.everyNMinutesTimeInterval": "Toutes les {value} {value, plural, one {minute} many {minutes} other {minutes}}",
Expand Down
2 changes: 2 additions & 0 deletions x-pack/plugins/translations/translations/ja-JP.json
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,8 @@
"coloring.dynamicColoring.rangeType.label": "値型",
"coloring.dynamicColoring.rangeType.number": "数字",
"coloring.dynamicColoring.rangeType.percent": "割合(%)",
"coloring.colorMapping.terms.otherBucketLabel":"その他",
"coloring.colorMapping.terms.emptyLabel": "(空)",
"console.helpPage.learnAboutConsoleAndQueryDslText": "{console}と{queryDsl}についてさらに詳しく",
"console.historyPage.itemOfRequestListAriaLabel": "リクエスト:{historyItem}",
"console.settingsPage.refreshInterval.everyNMinutesTimeInterval": "{value}{value, plural, other {分}}毎",
Expand Down
2 changes: 2 additions & 0 deletions x-pack/plugins/translations/translations/zh-CN.json
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,8 @@
"coloring.dynamicColoring.rangeType.label": "值类型",
"coloring.dynamicColoring.rangeType.number": "数字",
"coloring.dynamicColoring.rangeType.percent": "百分比",
"coloring.colorMapping.terms.otherBucketLabel":"其他",
"coloring.colorMapping.terms.emptyLabel": "(空)",
"console.helpPage.learnAboutConsoleAndQueryDslText": "了解 {console} 和 {queryDsl}",
"console.historyPage.itemOfRequestListAriaLabel": "请求:{historyItem}",
"console.settingsPage.refreshInterval.everyNMinutesTimeInterval": "每 {value} {value, plural, other {分钟}}",
Expand Down

0 comments on commit c828ff4

Please sign in to comment.