<blockquote style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;" class="gmail_quote">The word "recommended" is bolded for a reason. If it said "required"<br>
then we'd be having a different kind of discussion.<br></blockquote>
<br>Yes, but it also says that "if incorporated" I have to support the core WP implementation and that is what was pointed out to me in my last review. Technically though, as you have said, this functionality is not a "Custom Image Header". <br>
<br>I have the capability to define a header background like Kubrick did (and I call it a header background). In one way or the other this is still a customizable header, and the theme is tagged with "custom-header" because of it. AFAIK the tags don't distinguish between custom image header and custom header background, and the review team doesn't distinguish either. A reviewer probably sees "custom-header", then looks for a call to add_custom_image_header(). So maybe this is something to note in the review guidelines, then? Or maybe the text for this particular guideline has to be amended?<br>
<br><div class="gmail_quote">On Wed, Sep 7, 2011 at 4:14 PM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com">otto@ottodestruct.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Wed, Sep 7, 2011 at 6:10 PM, Sayontan Sinha <<a href="mailto:sayontan@gmail.com">sayontan@gmail.com</a>> wrote:<br>
> So a question for the reviewers - the review guidelines state:<br>
><br>
>> Theme is recommended to incorporate the following WordPress core<br>
>> functionality, but is not required to do so. However, if incorporated,<br>
>> functionality must support the core WordPress implementation:<br>
>><br>
>> add_custom_image_header().<br>
><br>
><br>
> Is it okay to not use the above function with a custom header then?<br>
<br>
</div>The word "recommended" is bolded for a reason. If it said "required"<br>
then we'd be having a different kind of discussion.<br>
<br>
If you're doing something like allowing the user to select tiled<br>
images and implementing them through CSS backgrounds, then that's not<br>
really a "custom image header", as I see it. That's more like a<br>
"custom header background" or something. Maybe the wording could be<br>
better in your theme, to distinguish it? Wording is tricky that way.<br>
<div><div></div><div class="h5"><br>
-Otto<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Sayontan Sinha<br><a href="http://mynethome.net" target="_blank">http://mynethome.net</a> | <a href="http://mynethome.net/blog" target="_blank">http://mynethome.net/blog</a><br>
--<br>Beating Australia in Cricket is like killing a celebrity. The death gets more coverage than the crime.<br><br>