Skip to content

[Form] Documented the choice_self_translation option of LanguageType #12750

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

Merged
merged 1 commit into from
Dec 4, 2019

Conversation

javiereguiluz
Copy link
Member

Fixes #12745.

@@ -74,6 +75,19 @@ If this option is ``true``, the choice values use the `ISO 639-2 alpha-3`_
three-letter codes (e.g. French = ``fra``) instead of the default
`ISO 639-1 alpha-2`_ two-letter codes (e.g. French = ``fr``).

choice_self_translation
Copy link
Member

Choose a reason for hiding this comment

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

By only looking at the option name I would not really know what this is supposed to do. Can we think of something more self-explanatory?

Copy link
Member Author

Choose a reason for hiding this comment

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

I agree with you. I've updated the description. Is it more clear now? Thanks.

Copy link
Member

@xabbuh xabbuh Dec 4, 2019

Choose a reason for hiding this comment

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

I was rather thinking about renaming the option. But sadly I cannot come up with a better name. 😞

Copy link
Member

Choose a reason for hiding this comment

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

But we can discuss that separately.

@javiereguiluz javiereguiluz merged commit 7361fdb into symfony:master Dec 4, 2019
@javiereguiluz javiereguiluz deleted the fix_12745 branch December 4, 2019 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Form] Allow to translate each language into its language in LanguageTy…
4 participants