<div dir="ltr">Thanks Otto,<div><br></div><div>1. The image in question for the footer, is a logo icon or logo + text such as</div><div><a href="http://www.megathemes.com/wp-content/themes/at/images/at.png">http://www.megathemes.com/wp-content/themes/at/images/at.png</a></div>
<div><br></div><div>2. Ok so for the background image if not integrated can it use custom background functionality until the core is integrated? so i would know how to develop it, thanks.</div><div><br></div><div>3. I thought the read more comments link was dissallowed as mentioned in my theme review but anyway it has both read more and comments link</div>
<div><br></div><div>4. widget inception it is, silly or not, i saw some themes in wordpress that use it (developing their own blocks, etc, platform theme for example), what would be the guidelines for this type of thing.</div>
<div><br></div><div>Thanks again for your time.<br><br><div class="gmail_quote">On Thu, Sep 8, 2011 at 9:50 PM, Otto <span dir="ltr">&lt;<a href="mailto:otto@ottodestruct.com">otto@ottodestruct.com</a>&gt;</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 Thu, Sep 8, 2011 at 1:32 PM, Robert Ambartsumov &lt;<a href="mailto:robert6292@gmail.com">robert6292@gmail.com</a>&gt; wrote:<br>

&gt; 1. Sanitazation/Validation of data if input has js validation - should there<br>
&gt; be php validation as well (for example if js is disabled)<br>
<br>
</div>YES. Always validate the data on the backend, period. Never trust any<br>
input coming from the browser. Standard security rule.<br>
<div class="im"><br>
&gt; 2. Theme image logos, are allowed by default to be as a theme name or for<br>
&gt; example with additional theme icon appended to the theme name in the form of<br>
&gt; an image - is this allowed?<br>
&gt; chipbennett said, &quot;I agree: default logo should not be promotional in<br>
&gt; nature&quot; but what does promotional mean?<br>
<br>
</div>Promotional as in advertising for your company, or a product, or<br>
anything not directly related to the theme in some way. You&#39;d be<br>
surprised at how people try to sneak advertising spam in... Basically,<br>
the theme shouldn&#39;t be a walking billboard for your theme development<br>
company, or for cheap viagra, or anything else by default.<br>
<div class="im"><br>
&gt; 3. footer link to the author of the theme, can this link be as an image, and<br>
&gt; the image will be as a small icon or perhaps simply the anchor text of the<br>
&gt; company/site name? or this link must be in pure text? please clarify<br>
<br>
</div>A small image, if suitably discreet, might be acceptable. Depends on<br>
the individual case. Different people see different things as &quot;spam&quot;.<br>
Realistically, it&#39;s hard to imagine a suitable image for that sort of<br>
thing, with maybe the exception of a logo or something.<br>
<br>
Any image included must be *in the theme itself*, not pulled from an<br>
external site.<br>
<div class="im"><br>
&gt; 4. background wp core functions, if the theme trys to provide additional<br>
&gt; functionality as in the form of pre-defined image patterns that can be<br>
&gt; selected by the user, so can it be integrated NOT by wp core background<br>
&gt; functions ?<br>
<br>
</div>The core custom header functions are capable of this, actually, but<br>
the background one is not. This seems like an oversight to me, perhaps<br>
you should make a ticket and try to get core support added for this.<br>
<div class="im"><br>
&gt; 5. theme favicon default image is changable in theme options, is this<br>
&gt; allowed?<br>
<br>
</div>I don&#39;t know of any guideline specifically disallowing it, although I<br>
think this might more realistically be plugin-territory. A person<br>
wouldn&#39;t want their favicon to disappear when they changed themes.<br>
<div class="im"><br>
&gt; 6. data test for posts with empty title, the post has a read more link that<br>
&gt; is not being generated by wp, this is a custom read more link that can also<br>
&gt; be disabled in theme options, is this enough for this check to pass? (the<br>
&gt; check that posts with empty titles must have a link to the post).<br>
<br>
</div>Probably, but I would also expect some more obvious linkage. Maybe a<br>
&quot;Read comments&quot; link or something along those lines? Any way for the<br>
user to see the single post page will do.<br>
<div class="im"><br>
&gt; 7. Assuming my themes use advanced widgetized blocks, the entire theme is<br>
&gt; litteraly widgetized in a very advanced way, would I also need to use the<br>
&gt; default wp core functions for widgets or not use them, or use both mine and<br>
&gt; wp core widget functions? the wp core widget functions i think wont be<br>
&gt; enough for this because I plan to create a widget block that inside that<br>
&gt; widget block there will be more widgets so it will be widget 3 layers deep<br>
&gt; kind of.<br>
<br>
</div>Widget Inception? Now I think you&#39;re just being silly.<br>
<font color="#888888"><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>
</font></blockquote></div><br></div></div>