<div dir="ltr"><div>Thanks Ulrich.<br></div>Handbook is looking good.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 11, 2014 at 9:28 PM, Ulrich Pogson <span dir="ltr"><<a href="mailto:grapplerulrich@gmail.com" target="_blank">grapplerulrich@gmail.com</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">This is the section on child theme support. <a href="https://make.wordpress.org/themes/handbook/guidelines/including-stylesheets-scripts/#child-theme-support" target="_blank">https://make.wordpress.org/themes/handbook/guidelines/including-stylesheets-scripts/#child-theme-support</a></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 11 October 2014 17:48, Srikanth Koneru <span dir="ltr"><<a href="mailto:tskk79@gmail.com" target="_blank">tskk79@gmail.com</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>First part is only recommended because themes are only required to be child friendly to extent that importing parent theme's stylesheet in child theme's stylesheet should work.<br><br></div>Second part is actually somewhere in the guidelines as required.<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 11, 2014 at 8:49 PM, Bego Mario Garde <span dir="ltr"><<a href="mailto:pixolin@yahoo.com" target="_blank">pixolin@yahoo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
I have another question, this time regarding the usage of<br>
<br>
  if( !function_exists()) {…}<br>
<br>
for non-core theme-functions.<br>
<br>
Otto (Samuel Wood) explained very well on<br>
<a href="http://wordpress.stackexchange.com/questions/111299/why-use-if-function-exists" target="_blank">http://wordpress.stackexchange.com/questions/111299/why-use-if-function-exists</a><br>
<br>
"The if function_exists approach allows for a child theme to override<br>
the function definition by simply defining the function themselves.<br>
Since child theme's functions.php files load first, then they will<br>
define the function first and the parent's definition will not get loaded."<br>
<br>
Chip Bennet defined on<br>
<a href="http://www.chipbennett.net/2011/04/20/a-guide-to-reviewing-themes-for-the-wordpress-theme-repository/" target="_blank">http://www.chipbennett.net/2011/04/20/a-guide-to-reviewing-themes-for-the-wordpress-theme-repository/</a><br>
<br>
<br>
"No core WordPress functions introduced prior to one previous, major<br>
WordPress release are wrapped in function_exists() conditionals."<br>
<br>
Yet I miss this being mentioned in the theme review guidelines nor do I<br>
know whether wrapping functions is only *recommended* or even *required*.<br>
<br>
Could someone please explain how I should deal with it?<br>
<br>
Thank you, Bego (@pixolin)<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>
</blockquote></div><br></div>
</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>