[theme-reviewers] I need help with this review

[Emil Uzelac]

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/a7bd38cd/attachment.htm>