Twenty Twelve - localization of next and previous posts

[pavelevap]

There is no need to translate this (see attached patch)?

<span class=\"meta-nav\">&larr;</span>
<span class=\"meta-nav\">&rarr;</span>

It would make sense only together with %title in one sentence?

[lancewillett]

Yes, it's needed for RTL languages where the arrows point the other direction.

[johnbillion]

Would those links be better off with context and no markup in the localised string? Eg:

previous_post_link( '%link', '<span class="meta-nav">' . _x( '&larr;', 'Previous post link', 'twentytwelve' ) . '</span> %title' );

[nacin]

Replying to johnbillion:

Would those links be better off with context and no markup in the localised string? Eg:

previous_post_link( '%link', '<span class="meta-nav">' . _x( '&larr;', 'Previous post link', 'twentytwelve' ) . '</span> %title' );

When we last had to translate something for previous_post_link() or next_post_link() — in Twenty Ten — we did this.

[SergeyBiryukov]

twentytwelve_content_nav() has a similar instance:
​http://core.trac.wordpress.org/browser/trunk/wp-content/themes/twentytwelve/functions.php?rev=22020#L249

[lancewillett]

22008.patch looks great to me, the other cases in the theme have English words tied to the arrow entities. In this case, it lives by itself so the _x() wrapper with context for translators is a nice improvement.

[lancewillett]

In [22035]:

Twenty Twelve: better i18n for next and previous post arrow entities in single.php, props SergeyBiryukov. Fixes #22008.