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 ISO-8859-1 encoding for i18n files #52

Closed
wants to merge 1 commit into from
Closed

Conversation

RandomByte
Copy link
Member

As expected by the UI5 Tooling

As expected by the UI5 Tooling
@RandomByte RandomByte requested a review from matz3 July 3, 2019 15:33
@matz3
Copy link
Member

matz3 commented Jul 16, 2019

Encoding conversion not required based on new concept: SAP/ui5-tooling#168

@matz3 matz3 closed this Jul 16, 2019
@matz3
Copy link
Member

matz3 commented Jul 16, 2019

@RandomByte or do we want to use normal UTF-8 then?

@RandomByte
Copy link
Member Author

Well, UTF8 would be nice but bloats the ui5.yaml a bit.

I don't like the current content of the _de file (containing unicode escapes) since they are hard to read.

I think I'm in favor of using normal UTF-8, without escapes and the additional ui5.yaml configuration.

@RandomByte RandomByte deleted the latin1-encoding branch July 22, 2019 13:39
@RandomByte
Copy link
Member Author

Follow up: #54

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.

2 participants