search

WPBuddy / largo

A WordPress framework for news websites. Finely-crafted by INN and expertly-honed and maintained by the technology team at WP Buddy.
http://largo.wpbuddy.co
GNU General Public License v2.0
170 stars 83 forks source link

Improve translatability #1592

Open benlk opened 5 years ago

benlk commented 5 years ago

The following issues were raised when running grunt pot for 0.6.1:

A long list - [ ] Warning: The string "Copyright %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/header-footer.php:69) - [ ] Warning: The string "%s is a member of the Institute for Nonprofit News" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/header-footer.php:85) - [ ] Warning: The string "Largo has been updated! Please visit the update page to apply a required database update." contains placeholders but has no "translators:" comment to clarify their meaning. (inc/update.php:873) - [ ] -Warning: The string "Please visit the update page to apply required Largo updates before editing Theme Options." contains placeholders but has no "translators:" comment to clarify their meaning. (inc/update.php:1166) - [ ] Warning: The string "Default (%s)" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/sidebars.php:182) - [ ] W arning: The string "" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/pagination.php:128) - [ ] Warning: The string "" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/pagination.php:135) - [ ] Warning: The string "" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/pagination.php:145) - [ ] Warning: The string "" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/pagination.php:152) - [ ] Warning: The string "%s ago" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/post-tags.php:55) - [ ] Warning: The string "More from %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/post-tags.php:104) - [ ] Warning: The string "<%1$s class="post-category-link">%5$s%4$s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/related-content.php:271) - [ ] Warning: The string "<%1$s class="post-tag-link">%5$s%4$s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/related-content.php:292) - [ ] Warning: The string "Default: %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/post-metaboxes.php:179) - [ ] Warning: The string "Click here to change the default post template." contains placeholders but has no "translators:" comment to clarify their meaning. (inc/post-metaboxes.php:183) - [ ] Warning: The string "%sYou have not set a description for your site.%s Add a site description by visiting %sthe Largo Theme Options page%s." contains placeholders but has no "translators:" comment to clarify their meaning. (inc/widgets/largo-about.php:56) - [ ] Warning: Multiple placeholders should be ordered. (inc/widgets/largo-about.php:56) - [ ] Warning: The string "About %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/widgets/largo-about.php:103) - [ ] Warning: The string "" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/widgets/largo-follow.php:57) - [ ] Warning: The string "Follow %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/widgets/largo-follow.php:78) - [ ] Warning: The string "

You don't have any recent %s.

" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/widgets/largo-recent-posts.php:109) - [ ] Warning: The string "Select %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/widgets/largo-taxonomy-list.php:76) - [ ] Warning: The string "Read All Posts By %s" contains placeholders but has no "translators:" comment to clarify their meaning. (inc/byline_class.php:284) - [ ] Warning: The string "Monthly Archives: %s" contains placeholders but has no "translators:" comment to clarify their meaning. (archive.php:43) - [ ] Warning: The string "Yearly Archives: %s" contains placeholders but has no "translators:" comment to clarify their meaning. (archive.php:45) - [ ] Warning: The string "Search results for %s" contains placeholders but has no "translators:" comment to clarify their meaning. (search.php:12) - [ ] Warning: The string "Your search for %s returned " contains placeholders but has no "translators:" comment to clarify their meaning. (search.php:103) - [ ] Warning: The string "Enter the copyright and credit information to display in the footer. You can use %d to output the current year." contains placeholders but has no "translators:" comment to clarify their meaning. (options.php:136) - [ ] Warning: The string "© Copyright %d, " contains placeholders but has no "translators:" comment to clarify their meaning. (options.php:138) - [ ] Warning: The string "

About

" contains placeholders but has no "translators:" comment to clarify their meaning. (partials/author-bio-description.php:7) - [ ] Warning: The string "Apologies, but %s was not found. Perhaps searching will help." contains placeholders but has no "translators:" comment to clarify their meaning. (partials/content-not-found.php:9) - [ ] Warning: The string "Built with the Largo WordPress Theme from the Institute for Nonprofit News." contains placeholders but has no "translators:" comment to clarify their meaning. (partials/footer-boilerplate.php:26) - [ ] Warning: Multiple placeholders should be ordered. (partials/footer-boilerplate.php:26) - [ ] Warning: The string "
    • " contains placeholders but has no "translators:" comment to clarify their meaning. (partials/author-bio-social-links.php:56) - [ ] Success: POT file successfully generated!
    benlk commented 5 years ago

    This can be lumped under the https://github.com/INN/largo/issues/844 umbrella issue.

    benlk commented 5 years ago

    This issue may need to be updated from #844.

    benlk commented 5 years ago

    Since we're now past 0.6.2, anyone wanting to contribute this should:

    1. get set up with a local dev env containing Node.js and wp-cli
    2. run npm i to download Grunt and its dependencies
    3. run grunt pot to regenerate the translation files
    4. make note of any issues, such as the form of the list above
    5. fix the issues.