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

Split the comment #12193

Merged
merged 2 commits into from
Nov 22, 2018
Merged

Split the comment #12193

merged 2 commits into from
Nov 22, 2018

Conversation

garretthyder
Copy link
Contributor

@garretthyder garretthyder commented Nov 21, 2018

Description

Separate the comment so translator comment is in it's own block.

How has this been tested?

This is a change to comments so aside from loading to make sure nothing broke the test will be done as part of parsing.

Types of changes

Bug fix #12139 corrects the translator comments for i18n

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.

Seperate the comment so translator comment is in it's own block.
lib/client-assets.php Outdated Show resolved Hide resolved
Updated the comments to address @swisspidy's review.
Removed the redundant comment and switched to a single line comment block.
@swissspidy swissspidy added this to the 4.6 milestone Nov 21, 2018
@designsimply designsimply added the Internationalization (i18n) Issues or PRs related to internationalization efforts label Nov 21, 2018
@gziolo gziolo merged commit 93262fa into WordPress:master Nov 22, 2018
@garretthyder garretthyder deleted the 12139-fix-translator-comment branch November 23, 2018 17:51
youknowriad pushed a commit that referenced this pull request Nov 29, 2018
* Split the comment

Seperate the comment so translator comment is in it's own block.

* Updated comments to address review

Updated the comments to address @swisspidy's review.
Removed the redundant comment and switched to a single line comment block.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internationalization (i18n) Issues or PRs related to internationalization efforts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translator Comments
4 participants