[wp-trac] [WordPress Trac] #31979: Twenty Thirteen: Improving i18n for "Continue reading"

[WordPress Trac]

#31979: Twenty Thirteen: Improving i18n for "Continue reading"
---------------------------+--------------------
 Reporter:  dregad         |       Owner:
     Type:  enhancement    |      Status:  new
 Priority:  normal         |   Milestone:  4.3
Component:  Bundled Theme  |     Version:  4.1.1
 Severity:  normal         |  Resolution:
 Keywords:  has-patch      |     Focuses:
---------------------------+--------------------
Changes (by atimmer):

 * keywords:  needs-patch => has-patch


Comment:

 Replying to [comment:2 SergeyBiryukov]:
 > I guess we need one more `sprintf()` here:
 > {{{
 > the_content( sprintf(
 >       /* translators: %s: Name of current post */
 >       __( 'Continue reading %s <span class="meta-nav">→</span>',
 'twentythirteen' ),
 >       /* translators: %s: Name of current post */
 >       sprintf( _x( '%s', 'post name' ), the_title( '<span class="screen-
 reader-text">', '</span>', false ) )
 > ) );
 > }}}
 This solution doesn't solve the problem raised in the ticket, ''de'' would
 still be outside of the ''screen-reader-text'' span.

 [https://core.trac.wordpress.org/attachment/ticket/31979/31979.diff
 31979.diff] is simply the patch proposed in the OP applied to all three
 themes.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31979#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform