[theme-reviewers] I need help with this review

[Chip Bennett]

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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110422/f6b10b24/attachment-0001.htm>