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

Update eu.csv #734

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update eu.csv #734

wants to merge 2 commits into from

Conversation

alexgabi
Copy link

Please update basque language (eu)
Thanks

Please update basque language (eu)
Thanks
Copy link
Member

@JeromeMartinez JeromeMartinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the update proposal.
Below are some comments.

@@ -1,6 +1,6 @@
Language_ISO639;eu
Author_Email;[email protected]
Author_Name;Xabier Aramendi
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put the older name in Author_OldNames field.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot it. I will do it.

frame1; frame
frame2; frame
frame3; frame
bit1; bit1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems a regression, I doubt that you want "1" and so on displayed (several times, bit, Byte, channel...).
Remove the numbers.

@@ -1162,12 +1161,12 @@ SamplesPerFrame;
SamplingCount;Lagin zenbatekoa
SamplingRate;Laginketa neurria
Save;Gorde
ScanOrder;Mihaketa hurrenkera
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current translation is so wrong? (several times, when you remove completely a translation without a replacement)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of the more technical terms were not well translated and, in the case of those that are not usually used by non-expert users, the original term is usually maintained.

I made the corrections. Thank for your suggestions.
@JeromeMartinez
Copy link
Member

Superseeded by #928 (and same on me to have let this one open forever).

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