<div dir="ltr">I dunno, actually. It is an interesting question. How galleries display is part of the visual style of the theme. I don't see anything wrong with including masonry-enabled galleries in the theme, especially as some kind of optional thing.<div><br></div><div>Now, I would suggest to the theme author not to override the gallery shortcode itself, but to use the post_gallery filter instead, which will let them do essentially the same thing without tinkering with "shortcodes".<div><br></div><div>That said, you would want to make sure that the controls for galleries in WordPress still operate as you would expect them to, more or less. Obviously, something like masonry is probably going to change the meaning of "columns" and the like, but still, that should be mostly functional, to whatever extent makes sense.</div></div><div><br></div><div>But then again, anything you do here in a theme could be done in a plugin too, and said plugin could be made universal for any theme, so unless this particular gallery shortcode is deeply tied to the theme in some way, then it's probably better in a plugin.</div><div><br></div></div><div class="gmail_extra"><br clear="all"><div>-Otto</div>
<br><div class="gmail_quote">On Tue, Oct 7, 2014 at 6:18 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@uzelac.me" target="_blank">emil@uzelac.me</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Not silly at all :)</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">This would definitely fall under plugins. ​</div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Tue, Oct 7, 2014 at 6:11 PM, Ryan Cowles <span dir="ltr"><<a href="mailto:ryan.cowles@automattic.com" target="_blank">ryan.cowles@automattic.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr">Howdy,<div><br></div><div>I'm reviewing a theme that overwrites the native gallery shortcode with a custom Masonry gallery. Seems like a silly question, but I'm looking for confirmation - would this fall under plugin territory?</div><div><br></div><div>Thanks!</div><span><font color="#888888"><div>Ryan</div></font></span></div>
<br></div></div>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
<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>