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

[Reference] link translation DIC tags to components section #4247

Merged
merged 1 commit into from
Sep 22, 2014

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Sep 17, 2014

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets

The translation.loader and translation.dumper DIC tags link to the respective sections in the Translation component documentation (in addition to the changes done in #4166).

The `translation.loader` and `translation.dumper` DIC tags link to the
respective sections in the Translation component documentation.
.. seealso::

Learn how to :ref:`dump to custom formats <components-translation-custom-dumper>`
in the components section.
Copy link
Contributor

Choose a reason for hiding this comment

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

this is too generic? or you really mean component singular

Copy link
Member Author

Choose a reason for hiding this comment

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

Hm, don't you think it's clear enough with link directly pointing to the relevant part? Do you think that "[...] in the component's documentation" is better?

@wouterj wouterj added actionable Clear and specific issues ready for anyone to take them. Status: Reviewed and removed actionable Clear and specific issues ready for anyone to take them. labels Sep 21, 2014
@weaverryan
Copy link
Member

Looks good to me - just a little less duplication :). Thanks!

@weaverryan weaverryan merged commit 9790975 into symfony:2.3 Sep 22, 2014
weaverryan added a commit that referenced this pull request Sep 22, 2014
…on (xabbuh)

This PR was merged into the 2.3 branch.

Discussion
----------

[Reference] link translation DIC tags to components section

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |

The `translation.loader` and `translation.dumper` DIC tags link to the respective sections in the Translation component documentation (in addition to the changes done in #4166).

Commits
-------

9790975 link translation DIC tags to components section
@xabbuh xabbuh deleted the dic-translation-tags-references branch September 22, 2014 17:05
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.

4 participants