[theme-reviewers] Bundling Plugins

[Chip Bennett]

This is probably a good topic for a wider discussion.

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.

Chip

On Mon, May 2, 2011 at 12:16 AM, Emil Uzelac <emil at themeid.com> wrote:

> 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
> http://yoast.com/wordpress/breadcrumbs/ and I am not sure about
> http://themes.svn.wordpress.org/billions/1.1.2/core/plugins/seo-features.php.
> 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,
>
> Cheers,
> Emil
>
>
> *----*
> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:
> emil at themeid.com | http://themeid.com
> Make everything as simple as possible, but not simpler. - Albert Einstein
>
>
>
> On Sun, May 1, 2011 at 11:00 PM, Vicky Arulsingam <
> vicky.arulsingam at gmail.com> wrote:
>
>> What's the official word on themes bundling plugins? The theme I'm
>> currently reviewing (http://themes.trac.wordpress.org/ticket/3398) is
>> including Yoast Breadcrumbs and Add Meta Tags as part of the download
>> package.
>>
>>
>> -----
>> Vicky Arulsingam
>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110502/578a1956/attachment.htm>