Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Update da translations#26801

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fabpot merged 1 commit intosymfony:2.7fromc960657:translation-da
Apr 4, 2018
Merged

Update da translations#26801

fabpot merged 1 commit intosymfony:2.7fromc960657:translation-da
Apr 4, 2018

Conversation

@c960657
Copy link
Contributor

QA
Branch?2.7
Bug fix?yes
New feature?no
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed tickets
LicenseMIT
Doc PR

The Danish translations have several serious errors. This PR adjusts to official Danish orthography, and updates some texts for internal consistency and to better reflect the original English text.

@fabpot
Copy link
Member

Thank you@c960657.

@fabpotfabpot merged commitd0ea26b intosymfony:2.7Apr 4, 2018
fabpot added a commit that referenced this pull requestApr 4, 2018
This PR was merged into the 2.7 branch.Discussion----------Update da translations| Q             | A| ------------- | ---| Branch?       | 2.7| Bug fix?      | yes| New feature?  | no| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets || License       | MIT| Doc PR        |The Danish translations have several serious errors. This PR adjusts to official Danish orthography, and updates some texts for internal consistency and to better reflect the original English text.Commits-------d0ea26b Update da translations
<trans-unitid="19">
<source>This value is too long. It should have {{ limit }} character or less.|This value is too long. It should have {{ limit }} characters or less.</source>
<target>Værdien er for lang. Denskal have{{ limit }}bogstaver eller mindre.</target>
<target>Værdien er for lang. Denmå højst indeholde{{ limit }}tegn.</target>
Copy link
Member

Choose a reason for hiding this comment

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

The English message applies pluralization based on{{ limit }}. Should it be done also in Danish or would both variants be the same ?

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Yes, some Danish words, including “tegn”, are the same in singular and plural form. It sounds confusing, but somehow it isn't :-)

@c960657c960657 deleted the translation-da branchApril 5, 2018 10:57
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@stofstofstof left review comments

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@c960657@fabpot@stof@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp