[theme-reviewers] Child theme support in the theme...

[Justin Tadlock]

"All template tags and hooks used in a Theme are required to be 
implemented properly."
http://codex.wordpress.org/Theme_Review#Template_Tags_and_Hooks

If this guideline is followed, then your theme already supports child 
themes.

While we're on the subject though, I take issue with the "@import" part 
of the Child Theme Support draft section.  Some parent themes could very 
well auto-load the parent theme style.css file, so @import might not be 
necessary.


On 5/28/2013 8:45 AM, Edward Caissie wrote:
> As this particular guideline is prefaced with "This section is draft 
> only" the "REQUIREMENT" is not in fact actually mandatory, yet.
>
> A "draft" guideline is one that is being added and can be expected to 
> be read and implemented as is but should not be used as a 
> "show-stopper" for the review of a theme. It should most definitely be 
> noted on the theme trac ticket. Also, in most cases, "draft" 
> guidelines should still be adhered to but if one (or more) "draft" 
> guidelines are not met it is not reason enough to resolve a theme as 
> "not-approved".
>
> Edward Caissie
> aka Cais.
>
>
> On Tue, May 28, 2013 at 7:50 AM, Jason Gottschalk <jason at gottschalk.ca 
> <mailto:jason at gottschalk.ca>> wrote:
>
>     Yes child theme support is required. See
>     http://codex.wordpress.org/Theme_Review#Child_Theme_Support
>
>
>     On Tue, May 28, 2013 at 8:13 AM, priyanshu mittal
>     <priyanshu.mittal at gmail.com <mailto:priyanshu.mittal at gmail.com>>
>     wrote:
>
>         @jose . Recently I have developed a theme Appointment version
>         1.1 which is not-approved till date and the team review member
>         gave the following feedback  " *Themes are required to
>         facilitate the use of Child Themes. A "basic" Child Theme
>         (i.e. a style.css with Template header tag and @import() of
>         the Template style.css), when activated, should function
>         exactly as the Theme itself functions.
>         bloginfo('stylesheet_url') and bloginfo(stylesheet_directory)
>         were found in the file header.php. Use
>         get_stylesheet_directory_uri() instead.*"
>
>         I tried to ask regarding this in the same ticket but dint get
>         any replies from past 40 hrs.... Please Help...
>
>
>         On Tue, May 28, 2013 at 4:29 PM, Jose Castaneda
>         <jomcastaneda at gmail.com <mailto:jomcastaneda at gmail.com>> wrote:
>
>             From what I can recall it is recommended but not required.
>             I believe it was Chip that added something about that in
>             the Theme Review Codex. It was also discussed some time
>             ago. Can't really check at the moment though.
>
>             Sent from my iPhone
>
>             On May 28, 2013, at 3:53 AM, priyanshu mittal
>             <priyanshu.mittal at gmail.com
>             <mailto:priyanshu.mittal at gmail.com>> wrote:
>
>>             Is that mandatory to support child themes while making a
>>             custom wordpress.org <http://wordpress.org> theme. Also
>>             if yes How can i enable this support.. Or what changes do
>>             I need to make in style.css.
>>             _______________________________________________
>>             theme-reviewers mailing list
>>             theme-reviewers at lists.wordpress.org
>>             <mailto:theme-reviewers at lists.wordpress.org>
>>             http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>             _______________________________________________
>             theme-reviewers mailing list
>             theme-reviewers at lists.wordpress.org
>             <mailto:theme-reviewers at lists.wordpress.org>
>             http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
>         _______________________________________________
>         theme-reviewers mailing list
>         theme-reviewers at lists.wordpress.org
>         <mailto:theme-reviewers at lists.wordpress.org>
>         http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
>
>     _______________________________________________
>     theme-reviewers mailing list
>     theme-reviewers at lists.wordpress.org
>     <mailto: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/20130528/592e15ad/attachment-0001.html>