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?

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

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' );

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.

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

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.

In [22035]:

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