<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Tested in IE6, IE7, IE8, IE9 and could not replicate the message.</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">You are talking about:</font></font></font></div><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"><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
<span class="Apple-tab-span" style="white-space:pre"> </span><script type="text/javascript"> <br><span class="Apple-tab-span" style="white-space:pre"> </span>var $buoop = {} <br><span class="Apple-tab-span" style="white-space:pre"> </span>$buoop.ol = window.onload; <br>
<span class="Apple-tab-span" style="white-space:pre"> </span>window.onload=function(){ <br><span class="Apple-tab-span" style="white-space:pre"> </span> if ($buoop.ol) $buoop.ol(); <br><span class="Apple-tab-span" style="white-space:pre"> </span> var e = document.createElement("script"); <br>
<span class="Apple-tab-span" style="white-space:pre"> </span> e.setAttribute("type", "text/javascript"); <br><span class="Apple-tab-span" style="white-space:pre"> </span> e.setAttribute("src", "<a href="http://browser-update.org/update.js">http://browser-update.org/update.js</a>"); <br>
<span class="Apple-tab-span" style="white-space:pre"> </span> document.body.appendChild(e); <br><span class="Apple-tab-span" style="white-space:pre"> </span>} <br><span class="Apple-tab-span" style="white-space:pre"> </span></script> </blockquote>
</font></font></font></div><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">If this is for < IE7 nothing to worry about as IE6 is gone anyways.</font></font></font></div>
<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 Tue, Apr 19, 2011 at 5:44 PM, Curtis McHale <span dir="ltr"><<a href="mailto:curtis@curtismchale.ca">curtis@curtismchale.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
The theme I'm reviewing has JS that detects the browser version and tells them to update (<a href="http://browser-update.org/" target="_blank">http://browser-update.org/</a>). There is not a theme option to remove this function or information in the description.<div>
<br></div><div>What is the theme review stance on this feature?</div><div><br></div><div>There are also some list styles missing in comments. I don't think this is a reason for failure but should be noted and requested for update on the next version. Is that correct?</div>
<div><br></div><div>The theme options are strictly a page that suggests the user donate if they like the theme. I know that credit links are okay but I'm not sure about this instance.</div><div><br></div><div>The theme calls loop-archive.php through get_template_part but the file doesn't actually exist. Fortunately it defaults to loop.php but I'm wondering how this would be addressed. The theme works but I would think this should be corrected in the next version either by just using loop.php or by adding the file that is being called. Is that correct?</div>
<div><br></div><div>The theme doesn't support post_formats. I thought that they were supposed to at least declare it and support whichever they picked. I don't see this information on the Theme Review page so is this assumption incorrect?<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>
</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>