Translator context needed on a Customize.php string

[xibe]

In the file /trunk/wp-admin/customize.php, line 72:

<span class="preview-notice"><?php _e('You are previewing'); ?></span>
<strong class="theme-name"><?php echo $wp_customize->theme()->display('Name'); ?></strong>

The line "You are previewing" doesn't convey the fact that it is going to be followed by the theme's name; hence translators might miss a word (I almost did, had I not looked at the code).

I'd suggest either including the name within the string (as %s; better for RTL languages?), or adding a line of context.

My quick-n-dirty hack for a context.

Yes, this should probably be 'You are previewing %s', though that might be difficult given the UI there.

Replying to nacin:

Yes, this should probably be 'You are previewing %s'

Done in 20782.patch​. Tested in Firefox 12, Chrome 19, IE 8, Opera 11.64.

Replying to SergeyBiryukov:

Replying to nacin:

Yes, this should probably be 'You are previewing %s'

Done in 20782.patch​.

The issue is when the string gets split. For example, "You are %s previewing" -- now what?

Well, if some language requires to split the string:

1. Currently there's no way do to that, so the translation wouldn't be flexible.
2. It looks a bit weird, but doesn't seem too bad (20782.split-example.png​). Or, if that's what we're trying to avoid, what would be a better UI solution in this case?

If the language doesn't require splitting the string, this won't be an issue, unless I'm missing something.

20782.split-example.png​ doesn't look bad at all.

In [20955]:

Theme Customizer: Add the theme name to the 'You are previewing' string in the left pane, using a placeholder. props SergeyBiryukov, xibe. fixes #20782.