Yeah the footer was already on my list just obvious enough that I didn't mention it.<br clear="all"><br>Curtis McHale<br>PH: 604.751.3482<br><a href="http://www.curtismchale.ca">http://www.curtismchale.ca</a><br>Linkedin: <a href="http://www.linkedin.com/in/curtismchale">http://www.linkedin.com/in/curtismchale</a><br>
Twitter: <a href="http://twitter.com/curtismchale">twitter.com/curtismchale</a><br><br>
<br><br><div class="gmail_quote">On Tue, Apr 19, 2011 at 4:07 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" face="arial, helvetica, sans-serif">note footer for <a href="http://codex.wordpress.org/Theme_Review#Credit_Links" target="_blank">http://codex.wordpress.org/Theme_Review#Credit_Links</a> there are two and it can be only one, also one is 404.<br>
</font><div><div class="im"><div><b><font color="#333333" face="arial, helvetica, sans-serif"><br></font></b></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 class="h5"><div class="gmail_quote">On Tue, Apr 19, 2011 at 6:03 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">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 style="white-space:pre-wrap"> </span><script type="text/javascript"> <br><span style="white-space:pre-wrap"> </span>var $buoop = {} <br><span style="white-space:pre-wrap"> </span>$buoop.ol = window.onload; <br>
<span style="white-space:pre-wrap"> </span>window.onload=function(){ <br><span style="white-space:pre-wrap"> </span> if ($buoop.ol) $buoop.ol(); <br><span style="white-space:pre-wrap"> </span> var e = document.createElement("script"); <br>
<span style="white-space:pre-wrap"> </span> e.setAttribute("type", "text/javascript"); <br><span style="white-space:pre-wrap"> </span> e.setAttribute("src", "<a href="http://browser-update.org/update.js" target="_blank">http://browser-update.org/update.js</a>"); <br>
<span style="white-space:pre-wrap"> </span> document.body.appendChild(e); <br><span style="white-space:pre-wrap"> </span>} <br><span style="white-space:pre-wrap"> </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><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><div class="gmail_quote"><div>On Tue, Apr 19, 2011 at 5:44 PM, Curtis McHale <span dir="ltr"><<a href="mailto:curtis@curtismchale.ca" target="_blank">curtis@curtismchale.ca</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div>
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></div></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></div></blockquote></div><br></div>
</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>