Improve text strings in Twenty Twenty-Five

[poena]

This ticket replaces ​GitHub issue 671 that was not solved in time before the release of the theme.

Translators have provided feedback that there are duplicate strings because
the theme adds translators contexts inconsistently.

Examples:
"Learn more" without context
"Learn more" with the context "Sample hero button"
"Learn more" with the context "Button text of intro section."

"Search" with the context "Button text. Verb"
"Search" with the context "Button text. Verb."

Which means that the text "Learn more" is added 3 times and "Search" two times.

---

In addition, after looking at some of the translations that have been added, it is my opinion that some of the pattern names and template part names could be improved to make them easier to translate.

For example, this pattern name clearly explains that it is a page template:
"Page template for the right-aligned blog" and this name explains that it is the search results:
"News blog search results"-
While this name "Right-aligned blog, search" may sound like it is referring to the word search as a verb.

"Vertical Header" has translators believing that "header" is referring to a heading/headline/title, and not an alternative site header template part.

---

There was also feedback shared on the WordPress Slack, saying that the theme is not following the
​Style guide for writing documentation.

The style guide for quotation marks is to use straight quotes ​https://make.wordpress.org/docs/style-guide/punctuation/quotation-marks/

Example of where curly quotes are used:

twentytwentyfive\patterns\banner-poster.php

<?php
echo wp_kses_post(
/* translators: This string contains the word "Stories" in four different languages with the first item in the locale's language. */
_x( '“Stories, <span lang="es">historias</span>, <span lang="uk">iсторії</span>, <span lang="el">iστορίες</span>”', 'Placeholder heading in four languages.', 'twentytwentyfive' )
);
?>

I have changed the context descriptions for problematic patterns. Now they should be consistent named across different patterns.

I have changed the context descriptions for problematic patterns. Now they should be consistent named across different patterns.

Yes, it's changed, the review from my side.

@vasilpichurov Are you able to address the feedback from @carolinan's review?

[burakkaptan]

@poena @karmatosed @desrosj
Could you please assign this ticket to me? I will work on it until I indicate otherwise.
Thank you!

[poena]

@burakkaptan You are welcome to work on it, but please read the ticket history and comments, there are already two contributors working it. Perhaps you can find a way to collaborate.

Contributors are not assigned to tickets in this repository, but this does not limit who can work on a ticket, everyone is welcome.

@hameem1102 The file you submitted is not a complete patch.

Please see these documents that can help you solve this issue:
​https://make.wordpress.org/core/handbook/tutorials/faq-for-new-contributors/
​https://make.wordpress.org/core/handbook/tutorials/working-with-patches/

​https://developer.wordpress.org/block-editor/reference-guides/block-api/block-patterns/
​https://developer.wordpress.org/themes/patterns/

​https://developer.wordpress.org/apis/internationalization/

[burakkaptan]

@poena I was under the impression that tickets are assigned to contributors. The FAQ page you shared clarified how things work. While I’ve read a lot of documentation related to the core, I hadn’t come across the FAQ page before. Thank you for pointing it out. I truly appreciate your guidance.

Let the other two contributors continue working on the issue.I’d be happy to work on other issues instead.

[jorbin]

Moving to 6.8 since the patch isn't ready but once this is ready we can consider a twenty-twentyfive only release.

[prathamesh24]

This patch improves the text strings in the Twenty Twenty-Five theme for better clarity and consistency.

[prathamesh24]

My previous patch focused on improving general text clarity,
but I see that additional improvements are needed for translation context and pattern names.
I will update the patch to include these changes, ensuring better clarity for translators
and improving pattern descriptions. Let me know if there are any other areas I should focus on!

[poena]

I don't understand. What previous patch? @prathamesh24 do you have more than one username?
62482-improve-text.patch is not a patch.
Please refer to the links in comment 9 about how to work with patches.

In Twenty Twenty-Five:

• Remove the unnecessary context from some text strings where the translatable text was the same but the context was different. This reduces the number of strings a translator needs to work with.
• Use straight double quotes and straight apostrophes, to follow the documentation standards.
• Update the name of headers, to make it easier for translators to understand that they are site headers not headings.
• Update the name of templates to make the naming more consistent. Example:

Title: Text-only blog, home is now Title: Text blog home to match Title: News blog home etc.

Trac ticket: https://core.trac.wordpress.org/ticket/62482

[audrasjb]

I left a few comments in the PR :)
The idea is to avoid escaped characters as much as possible, to simplify polyglot's work.

In Twenty Twenty-Five:

Remove the unnecessary context from some text strings where the translatable text was the same but the context was different. This reduces the number of strings a translator needs to work with.

Update the name of headers, to make it easier for translators to understand that they are site headers not headings.
Update the name of templates to make the naming more consistent.
Example:
Title: Text-only blog, home is now Title: Text blog home to match Title: News blog home etc.

Trac ticket: https://core.trac.wordpress.org/ticket/62482

[audrasjb]

@poena the last PR looks good to me. I think it's ready for commit.
However we'll need to get this committed before beta 1, so Monday 3 is the deadline for this ticket before being punted to 6.9 :)

[karmatosed]

I can take this on and get in for commit. It feels a shame to not have that happen when it's so ready.

[karmatosed]

In 59891:

Twenty Twenty-Five: Improves text strings.

This ticket replaces the issue not resolved in time before the release of the theme. This now takes the feedback of translators and iterates on that.

Props poena, desrosj, hameem1102, burakkaptan, jorbin, prathamesh24, audrasjb.
Fixes #62482.