Correct context for strings on taxonomy screens

[SergeyBiryukov]

We have four strings on taxonomy screens (see [29803] for reference):

_ex( 'Name', 'Taxonomy Name' )
_ex( 'Slug', 'Taxonomy Slug' )
_ex( 'Parent', 'Taxonomy Parent' )
_ex( 'Description', 'Taxonomy Description' )

• 'Taxonomy Name' should be 'term name', see WP_Terms_List_Table::get_columns()​.
• In the other three strings, the context can probably be removed, since we don't have in the list table, and it appears to be redundant. Originally introduced in [13077].

[SergeyBiryukov]

Hi andg, thanks for the patch! Just a note, attachments don't trigger email notifications, so it's a good idea to leave a comment whenever you upload something.

[SergeyBiryukov]

Apparently 'Parent' still needs a context, some languages translate it differently for terms and pages.

[SergeyBiryukov]

In 29821:

• Use correct context for 'Name' and 'Parent' strings on taxonomy screens.
• Remove unnecessary context for 'Slug' and 'Description' strings.

props andg.
fixes #29824.

[andg]

Replying to SergeyBiryukov:

Hi andg, thanks for the patch! Just a note, attachments don't trigger email notifications, so it's a good idea to leave a comment whenever you upload something.

Noted, thanks! ;)

[nacin]

I'd rather leave the context on these existing strings, no? Otherwise we're just breaking them for no apparent reason, and we may well need to add context back in the future. It's also possible that they need to know the situation in which it is used in order to translate it (which would be suitable as a description if the string doesn't need to be split, but context is also fine).

[SergeyBiryukov]

Replying to nacin:

I'd rather leave the context on these existing strings, no? Otherwise we're just breaking them for no apparent reason, and we may well need to add context back in the future.

We're actually merging them with the strings used in the list table​.

Unlike "Parent", which can be translated differently for taxonomies and post types, "Slug" appears to be a universal term. I'm fine with adding the context for it back, but it should be done in the list table as well, and the lack of it there was not reported before.

"Description" does not look like it needs a special context.

[pavelevap]

I agree with Sergey, there is no need for special context for "Slug" and "Description" in this case. Actually these strings are used for the same thing in many other places and bigger problem is in other context, for example for "Description" used in TinymCE, Media, etc.