"(Currently set to: %s)" string needs context

[SergeyBiryukov]

On the old Menus screen, the string is used for menu locations, and %s is a menu name.

In the Customizer, the same string is used for menus, and %s is a location slug.

On a related note, "(Current: %s)" is missing a translator comment for the placeholder (%s is a menu name).

[shahpranaf]

Updated Translation Context

[shahpranaf]

Hi SergeyBiryukov,

If I understood the bug correctly, the context is showing "Current Menu Location" but %s prints Menu name in Customizer.
I've updated the Context. Let me know if "Current menu name" is fine or "Current menu" is correct.

Thanks
Pranav Shah

[pavelevap]

@SergeyBiryukov: We are missing context for (Currently set to: %s), but on the other hand we have very similar string (Current: %s) for menu name. What about merging them? I do not see a difference, there is only "set to" part added?

[shahpranaf]

@SergeyBiryukov Any update on this?

Thanks

[ocean90]

I think in the Customizer we should show the name too, not the slug.

[ryankienstra]

@ocean90,
Attached is a patch, and here is a ​GitHub PR with the same edits.

This adds translator information for the menu location.
In data passed to a JavaScript file, it adds translator context after (Currently set to: %s).
And in the Customizer nav menu control, it adds a translator comment to indicate the menu name.

[wonderboymusic]

In 35676:

Customizer i18n: provide translator context for current menu name and location.

Props ryankienstra.
Fixes #33431.

[SergeyBiryukov]

In 35722:

Customizer: Use correct context and translator comments for menu location strings.

See #33431.

[SergeyBiryukov]

Replying to ocean90:

I think in the Customizer we should show the name too, not the slug.

Yep, moved to a separate ticket: #34755.