[theme-reviewers] Another ticket

[Curtis McHale]

Yeah the footer was already on my list just obvious enough that I didn't
mention it.

Curtis McHale
PH: 604.751.3482
http://www.curtismchale.ca
Linkedin: http://www.linkedin.com/in/curtismchale
Twitter: twitter.com/curtismchale



On Tue, Apr 19, 2011 at 4:07 PM, Emil Uzelac <emil at themeid.com> wrote:

> note footer for http://codex.wordpress.org/Theme_Review#Credit_Links there
> are two and it can be only one, also one is 404.
> *
> *
> *----*
> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:
> emil at themeid.com | http://themeid.com
> Make everything as simple as possible, but not simpler. - Albert Einstein
>
>
>
> On Tue, Apr 19, 2011 at 6:03 PM, Emil Uzelac <emil at themeid.com> wrote:
>
>> Tested in IE6, IE7, IE8, IE9 and could not replicate the message.
>>
>> You are talking about:
>>
>>  <script type="text/javascript">
>>> var $buoop = {}
>>> $buoop.ol = window.onload;
>>>  window.onload=function(){
>>>  if ($buoop.ol) $buoop.ol();
>>>  var e = document.createElement("script");
>>>  e.setAttribute("type", "text/javascript");
>>>  e.setAttribute("src", "http://browser-update.org/update.js");
>>>  document.body.appendChild(e);
>>> }
>>> </script>
>>
>>
>> If this is for < IE7 nothing to worry about as IE6 is gone anyways.
>>
>> Emil
>>  *----*
>> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:
>> emil at themeid.com | http://themeid.com
>> Make everything as simple as possible, but not simpler. - Albert Einstein
>>
>>
>>
>> On Tue, Apr 19, 2011 at 5:44 PM, Curtis McHale <curtis at curtismchale.ca>wrote:
>>
>>> The theme I'm reviewing has JS that detects the browser version and tells
>>> them to update (http://browser-update.org/). There is not a theme option
>>> to remove this function or information in the description.
>>>
>>> What is the theme review stance on this feature?
>>>
>>> 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?
>>>
>>> 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.
>>>
>>> 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?
>>>
>>> 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?
>>>
>>> Curtis McHale
>>> PH: 604.751.3482
>>> http://www.curtismchale.ca
>>> Linkedin: http://www.linkedin.com/in/curtismchale
>>> Twitter: twitter.com/curtismchale
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110420/ca4436e6/attachment.htm>