Nothing wrong with Up-Sell Themes, for sure.<div><br></div><div>But: a *public-facing* up-sell ad? I agree with carolina n; there should be no such *public-facing* Up-Sell ads.</div><div><br></div><div>Chip<br><br><div class="gmail_quote">
On Fri, Apr 22, 2011 at 4:57 AM, Emil Uzelac <span dir="ltr">&lt;<a href="mailto:emil@themeid.com">emil@themeid.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<p>I don&#39;t see anything wrong with an upgrade to Pro version of the Theme, plugins do this all the time and unless I am off here this is acceptable. </p>
<p>This message&#39;s brevity courtesy of my Android.</p>
<p></p><blockquote type="cite"><div><div></div><div class="h5">On Apr 22, 2011 3:23 AM, &quot;carolina n&quot; &lt;<a href="mailto:myazalea@hotmail.com" target="_blank">myazalea@hotmail.com</a>&gt; wrote:<br><br>



<div>
 <br>
Theme: <a href="http://themes.trac.wordpress.org/ticket/3597" target="_blank">http://themes.trac.wordpress.org/ticket/3597</a><br>
 <br>
We have talked about not doing cursory reviews, but there are just too many errors, I dont feel that the author is giving the theme enough thought before he uploads it again,<br>
causing even more errors than the previous version....  Also the theme&#39;s option page is too advanced for me.<br>
I will sadly but without tearing more of my hair off, leave this to another reviewer.. because im exhausted..<br>
 <br>
Some notes:<br>
The theme displays both future and drafted posts... <br>
The author has fixed previous html validation errors but has also caused 57 new ones. <br>
He has fixed the php errors and the Layout looks acceptable when testing with the theme unit test data.<br>
 <br>
Parts of his option page is copied of <a href="http://blogatize.com" target="_blank">blogatize.com</a>, wich he also credits. But I&#39;m not sure whatever license that has, or where he got the code from. (I&#39;ve never used blogatize but it doesnt look like a place where you can get code examples...)<br>


The theme options work, data is somewhat vaildated. Functions are named differently, sometimes they are not prefixed at all.<br>
He is registering and enqueuing the same version of jquery that is bundled with wp but directly from google.<br>
 <br>
The default installation of the theme shows a public facing information box with how to upgrade to the pro version of the theme, but with this note:<br>
<em>(To remove this Widget login to your admin account, go to Appearance, then Widgets and drag new widgets into the Sidebar)</em><br>
The description of the theme goes:<br>
<em> iFeature: A Free WordPress Theme designed by CyberChimps.com (&lt;a href=&quot;</em><a href="http://cyberchimps.com/ifeaturepro" target="_blank"><em>http://cyberchimps.com/ifeaturepro</em></a><em>&quot; alt=&quot;Upgrade to iFeature Pro&quot;&gt;Upgrade to iFeature Pro&lt;/a&gt;)</em><br>


<em></em> <br>
-is this allowed?<br>
 <br>
 <br>
 <br>
 <br>
 <br>                                               </div>
<br></div></div>_______________________________________________<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><p></p>
<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>