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

πŸ§‘β€πŸ’» Translate template name if theme text domain can be loaded #163

Merged
merged 1 commit into from
Feb 3, 2023

Conversation

teppokoivula
Copy link
Contributor

This PR adds support for translating page/post template names in Blade templates, duplicating the translation behaviour from WP_Theme for FiltersTemplates.

Without this modification I couldn't find any truly sensible way to translate the visible labels for templates in the dropdown menu.

@teppokoivula
Copy link
Contributor Author

Hey @retlehs!

Just wondering if this might have a chance of making it into Acorn? If not - or if it's likely to take considerably longer - I'll start looking for an alternative approach πŸ™‚

Not being able to translate template names is not a major issue by any means, but it does make things somewhat unintuitive for our content editors.

Sorry for pinging you directly, by the way; not entirely sure what the process of handling PRs is like πŸ™πŸ™‚

@retlehs
Copy link
Member

retlehs commented Feb 3, 2023

@teppokoivula Thanks so much for this PR! My apologies on how long it's taken me to get back to you. Finally got around to testing this

@retlehs retlehs changed the title Translate template name if theme text domain can be loaded πŸ§‘β€πŸ’» Translate template name if theme text domain can be loaded Feb 3, 2023
@retlehs retlehs merged commit d9e1ec3 into roots:main Feb 3, 2023
@teppokoivula
Copy link
Contributor Author

Awesome β€”Β thanks for merging! πŸ™‚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants