<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">A common problem when themes do his is that they override the functionality of other plugins like Jetpack, however that can be avoided with just a simple check:<div><br></div><div><div>require_once( ABSPATH . 'wp-admin/includes/plugin.php' );</div><div>if( is_plugin_active( 'jetpack/jetpack.php' ) && in_array( 'tiled-gallery', Jetpack::get_active_modules() ) )</div><div><span class="Apple-tab-span" style="white-space:pre"> </span>return '';</div><div><br></div><div>Other than that I don't see any reason why themes shouldn't hack into the gallery short code, it's presentational after all.</div><div><br></div><div>Daniel</div><div><br><div><div>On Mar 26, 2013, at 8:26 PM, Jeremy Herve wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div style="font-family:arial,sans-serif;font-size:13px">Hi there,</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">I found that some themes in the repo currently include functions that customize the output of the Gallery shortcode.</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">I wonder if this is something we want to promote in the theme repo. Isn't it something that should remain in the plugin territory?</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">It is sometimes hard to troubleshoot gallery issues in the forums; you start by suggesting that they disable all image and gallery plugins, but that doest help. Unless you know the theme, you have no way to know if that theme actually includes functions that make changes to core WordPress features.</div>
<div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">What do you think? Is it something that could be added to the theme guidelines?</div><div style="font-family:arial,sans-serif;font-size:13px">
<br></div><div style="font-family:arial,sans-serif;font-size:13px">Cheers,</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Jeremy</div><div style="font-family:arial,sans-serif;font-size:13px">
<br></div><div style="font-family:arial,sans-serif;font-size:13px"><div>--</div><div>Jeremy Herve<div></div></div><div><a href="http://jeremyherve.com/" target="_blank">http://jeremyherve.com</a></div><div><a href="http://twitter.com/jeherve" target="_blank">http://twitter.com/jeherve</a></div>
</div></div>
_______________________________________________<br>theme-reviewers mailing list<br><a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>http://lists.wordpress.org/mailman/listinfo/theme-reviewers<br></blockquote></div><br></div></div></body></html>