[EUI] Fix EuiFilterButton badge i18n#109333
Conversation
|
The fix in Kibana LGTM. However, I wonder if we should fix it in the EUI library so any other consumers can take advantage of it? |
Yeah @constancecchen we'll want first change it in EUI and then update the i18n_eui_mapping file here after it's released during the upgrade. |
|
Ahh great calls! I actually did spike out a more opinionated change in EUI, basically splitting up into 2 different strings rather than trying to make a single string conditional. I think it will get translated better that way and possibly more smoothly in different languages, rather than trying to chop it up based on English. What do y'all think? I'll open up the PR for EUI, would love thoughts there :) |
|
That diff looks good to me, @constancecchen! |
|
Closing in favor of elastic/eui#5061 |
Summary
closes elastic/eui#4786
I picked up a super simple EUI issue to familiarize myself more with EUI's i18n process and relationship with Kibana's i18n :)
I think this should solve the problem described in the linked issue above, but let me know if not!
Checklist