Yeah, I think this is a sticky subject. I'm not sure I'm comfortable with it honestly. It's one thing to offer your own custom theme options but it's another thing altogether to bundle, in most cases, someone else's plugin. Not sure how I feel about that.<div>
<br></div><div>James<br><br><div class="gmail_quote">On Mon, May 2, 2011 at 8:01 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
This is probably a good topic for a wider discussion.<div><br></div><div>My initial thought is that *simple* Plugin functionality (e.g. breadcrumbs) being *properly incorporated* (i.e. not simply being bundled, and with proper function_exists checking for the original) is acceptable. But at some point, the complexity of the Plugin (e.g. Yoast SEO) should preclude it from being incorporated into the Theme.</div>
<div><br></div><div><font color="#888888">Chip</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">On Mon, May 2, 2011 at 12:16 AM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">If the plugin is properly implemented there should be no problems. For example Coraline Theme implemented Theme options based on a plugin, so it's OK. Yoast Breadcrumbs are still current <a href="http://yoast.com/wordpress/breadcrumbs/" target="_blank">http://yoast.com/wordpress/breadcrumbs/</a> and I am not sure about <a href="http://themes.svn.wordpress.org/billions/1.1.2/core/plugins/seo-features.php" target="_blank">http://themes.svn.wordpress.org/billions/1.1.2/core/plugins/seo-features.php</a>. Now there is one thing to pay attention to and that is if they can be disabled (turned off) from i.e. Theme Options, just in case that user wants to use something else. I am sure that breadcrumb will not cause issues, however meta plugin could if the user installs any other SEO plugin. If one is "hardcoded" and other one installed as "real" plugin from plugins they could affect each other and or create double meta tags,</font></font></font><div>
<font color="#333333" face="arial, helvetica, sans-serif"><br></font></div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"></font></font></font><font color="#333333" face="arial, helvetica, sans-serif">Cheers,</font></div>
<div><font color="#333333" face="arial, helvetica, sans-serif">Emil<br></font><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div>
<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><br clear="all"></font></font></font><div><strong><font face="arial, helvetica, sans-serif">----</font></strong></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: <a href="tel:224-444-0006" value="+12244440006" target="_blank">224-444-0006</a> | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br><div class="gmail_quote"><div>On Sun, May 1, 2011 at 11:00 PM, Vicky Arulsingam <span dir="ltr"><<a href="mailto:vicky.arulsingam@gmail.com" target="_blank">vicky.arulsingam@gmail.com</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
What's the official word on themes bundling plugins? The theme I'm currently reviewing (<a href="http://themes.trac.wordpress.org/ticket/3398" target="_blank">http://themes.trac.wordpress.org/ticket/3398</a>) is including Yoast Breadcrumbs and Add Meta Tags as part of the download package.<br clear="all">
<br><br>-----<div>Vicky Arulsingam</div><font color="#888888"><br>
</font><br></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></div>
<br>_______________________________________________<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></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>