<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">done <a href="http://themes.trac.wordpress.org/ticket/3597">http://themes.trac.wordpress.org/ticket/3597</a><br clear="all"></font></font></font><div>
<strong><font face="arial, helvetica, sans-serif">----</font></strong></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br><div class="gmail_quote">On Fri, Apr 22, 2011 at 4:41 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@themeid.com">emil@themeid.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Sounds good to me.</font></font></font><div><div class="im"><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><br clear="all">
</font></font></font><div>
<strong><font face="arial, helvetica, sans-serif">----</font></strong></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br></div><div><div></div><div class="h5"><div class="gmail_quote">On Fri, Apr 22, 2011 at 4:39 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I would note exactly the same comment as the reviewer. I think such a public-facing up-sell ad, even though removable, is too far. It is also needless. Up-sell ads should be confined to the admin UI.<div><br></div><div><font color="#888888">Chip</font><div>
<div></div><div><br>
<br><div class="gmail_quote">On Fri, Apr 22, 2011 at 4:36 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Once the widget is added the button will go away.</font></font></font><div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><span style="color:rgb(34, 34, 34);font-size:12px">(To remove this Widget login to your admin account, go to Appearance, then Widgets and drag new widgets into the Sidebar)</span></font></font></font></div>
<div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><span style="color:rgb(34, 34, 34);font-size:12px"><br></span></font></font></font></div></div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><span style="color:rgb(34, 34, 34)">Anything optional and stuff that can be removed with ease do not cause any problems, I think. </span></font></font></font></div>
<div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><span style="color:rgb(34, 34, 34)"><br></span></font></font></font></div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><span style="color:rgb(34, 34, 34)">Cheers,</span></font></font></font></div>
<div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><span style="color:rgb(34, 34, 34)">Emil</span></font></font></font></div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><br clear="all">
</font></font></font><div><strong><font face="arial, helvetica, sans-serif">----</font></strong></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<font color="#888888">
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div></font><div><div></div><div><br>
<br><br><div class="gmail_quote">On Fri, Apr 22, 2011 at 8:20 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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><font color="#888888">Chip</font><div>
<div></div><div><br><br><div class="gmail_quote">
On Fri, Apr 22, 2011 at 4:57 AM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>I don'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's brevity courtesy of my Android.</p>
<p></p><blockquote type="cite"><div><div></div><div>On Apr 22, 2011 3:23 AM, "carolina n" <<a href="mailto:myazalea@hotmail.com" target="_blank">myazalea@hotmail.com</a>> 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'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'm not sure whatever license that has, or where he got the code from. (I'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 (<a href="</em><a href="http://cyberchimps.com/ifeaturepro" target="_blank"><em>http://cyberchimps.com/ifeaturepro</em></a><em>" alt="Upgrade to iFeature Pro">Upgrade to iFeature Pro</a>)</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" 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" 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" 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" 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>
</blockquote></div><br>