Re: wp_link_pages()<div><br></div><div>We should defer to the developer's design aesthetic wherever possible. As long as wp_link_pages() is output in the single-post view, it should be fine.</div><div><br></div><div>Re: internationalization</div>
<div><br></div><div>The example you posted would be almost unintelligible to some translators, the way it is split apart. For correct implementation/best-practice, see Otto's post:</div><div><a href="http://ottopress.com/2012/internationalization-youre-probably-doing-it-wrong/">http://ottopress.com/2012/internationalization-youre-probably-doing-it-wrong/</a></div>
<div><br></div><div>I would require correct implementation (i.e. using sprintf() in this case), with an example for how to implement.</div><div><br></div><div>Thanks,</div><div><br></div><div>Chip<br><br><div class="gmail_quote">
On Sat, Mar 17, 2012 at 3:03 PM, Konstantin Obenland <span dir="ltr"><<a href="mailto:obenland@gmx.de">obenland@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">Hello,<div><br></div><div>it's me again, with yet another question. :/</div><div><br></div><div>I'd really appreciate a second opinion on this ticket (<a href="http://themes.trac.wordpress.org/ticket/6868" target="_blank">http://themes.trac.wordpress.org/ticket/6868</a>) if anyone gets a chance. Am I being to strict here?</div>
<div><br></div><div>Can Themes that display the excerpt in index pages omit the usage of wp_link_pages()?</div><div><br></div><div>And what about this best practice:</div><div>Use format strings instead of string concatenation — <tt>sprintf(__('Replace %1$s with %2$s'), $a, $b);</tt> is always better than <tt>__('Replace ').$a.__(' with ').$b;</tt></div>
<div><br></div><div>Would this be required, recommended or optional?</div><div>Example:</div><div><div><?php _e( '<font color="#4f792d">This is a gallery about</font>', 'esplanade' ); ?></div><div><a href="<?php the_permalink(); ?>" title="<?php the_title_attribute(); ?>" rel="bookmark"><?php the_title(); ?></a></div>
<div><?php _e( '<font color="#4f792d">and contains</font>', 'esplanade'); ?></div><div><?php echo count( get_posts( array( 'numberposts' => -1, 'post_type' => 'attachment', 'post_mime_type' => 'image', 'post_parent' => get_the_ID() ) ) ); ?></div>
<div><?php _e( '<font color="#4f792d">images</font>', 'esplanade'); ?>.</div></div><div><br></div><div>Thanks for your patience,</div><div>Konstantin</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>