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

Removed Translation component documentation#12861

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

Conversation

wouterj
Copy link
Member

While translation is one of the components that is used by other applications quite a lot, most of the components docs was about general translator usage (that also applies to usage in the framework). Setting up the translation component isn't that difficult.

This PR moves all relevant docs from the components doc to the main guide. Much of this was duplicated in both, so that proves that this move is a usefull thing to do. 2 articles have been completely removed: Creating your custom message formatter and format. A custom message formatter seems really advanced usage. The Translation component provides so many formats that I think creating a custom format is also quite advanced. If it turns out this is not true, we can rest assured that they still exist in GIT history so we can quickly add them back.

Please note that the main translator guide is a really ugly structured document. However, let's only do the rewriting in Symfony 4.3+, as lots has changed (flex, ICU message format, etc) between 3.x and 4.x.

Copy link
Member

@javiereguiluzjaviereguiluz left a comment

Choose a reason for hiding this comment

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

Wouter, thanks a lot for this. It's a great work as always. I left some comments, but they are less important than they may look. Cheers!

@@ -0,0 +1,44 @@
The XLIFF format
Copy link
Member

Choose a reason for hiding this comment

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

I know this article explains something very specific ... but creating a new tiny article of just 44 lines is what we're trying to avoid recently. Couldn't we add this in a section (or sidebar) of the main translation article?

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

I somewhat agree, but I don't think this matches our 80-20 rule. XLIFF formats have some specialized features, notes being one that is supported by Symfony. I don't think XLIFF notes belong to the 80% use cases of translation in a Symfony application. And if they do, it would target the translator more than a PHP developer (as notes are just for usage in the XLIFF editor AFAICS).

@wouterj
Copy link
MemberAuthor

Thanks@javiereguiluz. I've now updated the PR based on your suggestions. I've updated the docs to not talk much about message domains inaf22e56 . I agree that's its a premium feature that might easily be overused and confusing, so we should not talk about it too much in the docs :)

@wouterjwouterjforce-pushed therestructure-translation-component branch fromaf22e56 toeb7158bCompareJanuary 31, 2020 18:23
wouterj added a commit to wouterj/symfony-docs that referenced this pull requestFeb 1, 2020
wouterj added a commit that referenced this pull requestFeb 1, 2020
* 4.3:  [#12861] Tweak translation articles for 4.3 component changes
wouterj added a commit that referenced this pull requestFeb 1, 2020
* 4.4:  [#12861] Tweaked translation docs for 4.4 component changes  [#12861] Tweak translation articles for 4.3 component changes
@wouterjwouterj merged commiteb7158b intosymfony:3.4Feb 1, 2020
wouterj added a commit that referenced this pull requestFeb 1, 2020
* 5.0:  [#12861] Tweaked translation docs for 4.4 component changes  [#12861] Tweak translation articles for 4.3 component changes
@wouterjwouterj deleted the restructure-translation-component branchFebruary 1, 2020 12:38
@wouterj
Copy link
MemberAuthor

I've merged this into all versions of the docs and tweaked some docs that were moved to the main guide to follow previous changes between versions inc0f9bfa and98da366

OskarStark reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@javiereguiluzjaviereguiluzjaviereguiluz approved these changes

@xabbuhxabbuhAwaiting requested review from xabbuhxabbuh is a code owner

Assignees
No one assigned
Projects
None yet
Milestone
3.4
Development

Successfully merging this pull request may close these issues.

3 participants
@wouterj@javiereguiluz@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp