<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Well there is no specific point where to stop a review, one of the heavy cases would be if there is a bug that does not allow us to see the Theme in a browser, or due to heavy SPAM. If Theme Check does not find anything and if there are no visible bugs I personally would move to codes, starting from i.e. header.php or sometimes straight to functions.php and beyond.</font></font></font><div>

<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Emil <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 Wed, Apr 20, 2011 at 10:24 PM, Curtis McHale <span dir="ltr">&lt;<a href="mailto:curtis@curtismchale.ca">curtis@curtismchale.ca</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

I did just take a look. Ultimately the only issue I found was the fact that titles in widgets cover other text making it not readable. It was my understanding that this would mean a theme is not approved. If that&#39;s in error let me know.<div>



<br></div><div>I also looked at the functions file and nothing is prefixed so it still shouldn&#39;t be approved but I will of course update the information once I hear back.</div><div><br></div><div>A further question - At what point do you stop a review? In this case I stopped once I found the issue with the widgets but upon further inspection I found more issues with the theme which I wouldn&#39;t have caught if Emil hadn&#39;t asked me for more information.<div>

<div></div><div class="h5"><br clear="all">

<br>Curtis McHale<br>PH: 604.751.3482<br><a href="http://www.curtismchale.ca" target="_blank">http://www.curtismchale.ca</a><br>Linkedin: <a href="http://www.linkedin.com/in/curtismchale" target="_blank">http://www.linkedin.com/in/curtismchale</a><br>

Twitter: <a href="http://twitter.com/curtismchale" target="_blank">twitter.com/curtismchale</a><br>

<br>
<br><br><div class="gmail_quote">On Wed, Apr 20, 2011 at 8:15 PM, Emil Uzelac <span dir="ltr">&lt;<a href="mailto:emil@themeid.com" target="_blank">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">



<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Can you take a look at the comment I left please?<br clear="all"></font></font></font><div><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><div class="gmail_quote">On Wed, Apr 20, 2011 at 9:45 PM, Curtis McHale <span dir="ltr">&lt;<a href="mailto:curtis@curtismchale.ca" target="_blank">curtis@curtismchale.ca</a>&gt;</span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Done I can take another one.<div><br clear="all"><br>Curtis McHale<br>PH: 604.751.3482<br><a href="http://www.curtismchale.ca" target="_blank">http://www.curtismchale.ca</a><br>Linkedin: <a href="http://www.linkedin.com/in/curtismchale" target="_blank">http://www.linkedin.com/in/curtismchale</a><br>







Twitter: <a href="http://twitter.com/curtismchale" target="_blank">twitter.com/curtismchale</a><br><br>
<br><br></div><div><div></div><div><div class="gmail_quote">On Wed, Apr 20, 2011 at 7:15 PM, Emil Uzelac <span dir="ltr">&lt;<a href="mailto:emil@themeid.com" target="_blank">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">

<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">All right Curtis here it is <a href="http://themes.trac.wordpress.org/ticket/3543" target="_blank">http://themes.trac.wordpress.org/ticket/3543</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"><div><div></div><div>On Wed, Apr 20, 2011 at 9:12 PM, Curtis McHale <span dir="ltr">&lt;<a href="mailto:curtis@curtismchale.ca" target="_blank">curtis@curtismchale.ca</a>&gt;</span> wrote:<br>







</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div>

I just finished a review and am ready for a new ticket.<br clear="all"><font color="#888888"><br>Curtis McHale<br>PH: 604.751.3482<br><a href="http://www.curtismchale.ca" target="_blank">http://www.curtismchale.ca</a><br>









Linkedin: <a href="http://www.linkedin.com/in/curtismchale" target="_blank">http://www.linkedin.com/in/curtismchale</a><br>

Twitter: <a href="http://twitter.com/curtismchale" target="_blank">twitter.com/curtismchale</a><br><br>
</font><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></div><br>
<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><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><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>