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

Use unicode ellipsis instead of 3 dots "..." in user-facing strings #9997

Merged
merged 1 commit into from Sep 9, 2021

Conversation

Kisty
Copy link
Contributor

@Kisty Kisty commented Sep 9, 2021

Breaking change

Proposed change

Replace 3-dots ellipsis with Unicode's ellipsis character.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@Misiu
Copy link
Contributor

Misiu commented Sep 9, 2021

I think we're not allowed to directly modify files with translations, but instead, the translations should be updated in https://lokalise.com/, according to the docs: https://developers.home-assistant.io/docs/translations.

@bramkragten
Copy link
Member

Yes, only src/translations/en.json can be changed from this repo.

@Kisty
Copy link
Contributor Author

Kisty commented Sep 9, 2021

OK, thanks for your help guys. How can I resolve this? I could send a CL with only en.json changing and do the other changes in Lokalise.

Can I edit all languages in Lokalise? Would I have to say I'm a translator of all languages, I guess?

@bramkragten
Copy link
Member

Yes, you should have access to all languages if you signup

@bramkragten bramkragten enabled auto-merge (squash) September 9, 2021 15:31
@bramkragten bramkragten merged commit aba9667 into home-assistant:dev Sep 9, 2021
@spacegaier
Copy link
Member

spacegaier commented Sep 9, 2021

I am not sure if we can consistently use the Unicode. I assume in the future new translation strings will popup that use regular dots plus the community translations will not be consistent neither (most users will not know the difference).

That is why I had removed the few Unicode usages we had a couple of months ago 😎 so that we are indeed consistent visually in the frontend.

@Kisty
Copy link
Contributor Author

Kisty commented Sep 10, 2021

I've gone through the translations in Lokalise and converted most of them to the single unicode character.

@spacegaier that's fair. Some translations did in fact have the single character already. I'll finish them up soon. Wasn't quite as easy as doing a search and replace in Lokalise!

@spacegaier
Copy link
Member

Yes, it will be consistent now, but once we add new translation keys in the future, the community translations in lokalize will most likely use three dots, as most users are not aware that there is a dedicated Unicode symbol for it. Then the inconsistencies will start again.

To remain consistent the Github action that pulls in the lokalize translations into the frontend repo would need some logic to do an automatic conversion.

@Kisty
Copy link
Contributor Author

Kisty commented Sep 10, 2021 via email

@github-actions github-actions bot locked and limited conversation to collaborators Sep 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants