[theme-reviewers] I need help with this review

[Emil Uzelac]

done http://themes.trac.wordpress.org/ticket/3597
*----*
*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 Fri, Apr 22, 2011 at 4:41 PM, Emil Uzelac <emil at themeid.com> wrote:

> Sounds good to me.
>
>  *----*
> *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 Fri, Apr 22, 2011 at 4:39 PM, Chip Bennett <chip at chipbennett.net>wrote:
>
>> 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.
>>
>> Chip
>>
>>
>> On Fri, Apr 22, 2011 at 4:36 PM, Emil Uzelac <emil at themeid.com> wrote:
>>
>>> Once the widget is added the button will go away.
>>> (To remove this Widget login to your admin account, go to Appearance,
>>> then Widgets and drag new widgets into the Sidebar)
>>>
>>> Anything optional and stuff that can be removed with ease do not cause
>>> any problems, I think.
>>>
>>> Cheers,
>>> 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 Fri, Apr 22, 2011 at 8:20 AM, Chip Bennett <chip at chipbennett.net>wrote:
>>>
>>>> Nothing wrong with Up-Sell Themes, for sure.
>>>>
>>>> But: a *public-facing* up-sell ad? I agree with carolina n; there should
>>>> be no such *public-facing* Up-Sell ads.
>>>>
>>>> Chip
>>>>
>>>>
>>>> On Fri, Apr 22, 2011 at 4:57 AM, Emil Uzelac <emil at themeid.com> wrote:
>>>>
>>>>> 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.
>>>>>
>>>>> This message's brevity courtesy of my Android.
>>>>>
>>>>> On Apr 22, 2011 3:23 AM, "carolina n" <myazalea at hotmail.com> wrote:
>>>>>
>>>>>
>>>>> Theme: http://themes.trac.wordpress.org/ticket/3597
>>>>>
>>>>> 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,
>>>>> causing even more errors than the previous version....  Also the
>>>>> theme's option page is too advanced for me.
>>>>> I will sadly but without tearing more of my hair off, leave this to
>>>>> another reviewer.. because im exhausted..
>>>>>
>>>>> Some notes:
>>>>> The theme displays both future and drafted posts...
>>>>> The author has fixed previous html validation errors but has also
>>>>> caused 57 new ones.
>>>>> He has fixed the php errors and the Layout looks acceptable when
>>>>> testing with the theme unit test data.
>>>>>
>>>>> Parts of his option page is copied of blogatize.com, 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...)
>>>>> The theme options work, data is somewhat vaildated. Functions are named
>>>>> differently, sometimes they are not prefixed at all.
>>>>> He is registering and enqueuing the same version of jquery that is
>>>>> bundled with wp but directly from google.
>>>>>
>>>>> 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:
>>>>> *(To remove this Widget login to your admin account, go to Appearance,
>>>>> then Widgets and drag new widgets into the Sidebar)*
>>>>> The description of the theme goes:
>>>>> * iFeature: A Free WordPress Theme designed by CyberChimps.com (<a
>>>>> href="**http://cyberchimps.com/ifeaturepro*<http://cyberchimps.com/ifeaturepro>
>>>>> *" alt="Upgrade to iFeature Pro">Upgrade to iFeature Pro</a>)*
>>>>> **
>>>>> -is this allowed?
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> 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
>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> 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
>>>
>>>
>>
>> _______________________________________________
>> 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/20110422/ff12b925/attachment.htm>