[theme-reviewers] Themes With Customizable Logos

[Edward Caissie]

Making it a strongly recommended suggestion and providing the code is about
all you can do ... but as its not a "specific" requirement, but one inferred
from other criteria it is still at the authors discretion to use.

I would also suggest including your points in this thread with the comments
to explain why what you are suggesting would be much better for the author
and the theme end-users.


Cais.

On Thu, Sep 2, 2010 at 12:02 PM, Chip Bennett <chip at chipbennett.net> wrote:

> On Thu, Sep 2, 2010 at 10:52 AM, Ryan Hellyer <ryan at pixopoint.com> wrote:
>
>> >> In this particular Theme, the documentation basically says, "A logo.psd
>> >> and independence.ttf is included in the psd folder. Edit the psd file,
>> save
>> >> as logo.gif and upload to the images folder".
>>
>> >> At a minimum, it's a recommend with respect to Custom Image Header
>> >> functionality. I'm leaning toward require.
>>
>> Using a custom header like that then adds a "custom header" option in
>> the admin panel, which the theme developer likely did not intend. So
>> it should not be a requirement IMO.
>>
>> Ryan,
>
>
> It's a case-by-case consideration.
>
> Take a look at the Theme in question: http://themereview.chipbennett.net
>
> That "Food Recipe" is the header logo. The Theme gives instructions for how
> to change it. Thus, it would seem that the developer intends for it to be
> customizable.
>
> So, I'm not so much concerned about whether or not the Theme developer
> "intended" for a "Custom Header" menu item to display in the users' back
> end, as I am concerned with whether or not the Theme developer intended for
> the header logo to be customizable.
>
> Again, these are the instructions he gives:
>
> A logo.psd and independence.ttf is included in the psd folder. Edit the psd
> file, save as logo.gif and upload to the images folder.
>
>
> That implies to me that the logo is intended to be customizable. The core
> functionality vastly improves the user experience with respect to uploading
> customized logos.
>
> All that it would take to implement the core feature would be:
>
> 1) include add_custom_image_header()
> 2) define the custom header image dimensions
> 3) define the default image
>
> Easy peasy. In fact, I'm going to put the code into a comment on the
> ticket, to show how easy it would be to implement.
>
> In the end, I don't see how this is all that much different from requiring
> wp_nav_menu() when a Theme is using wp_list_pages() as a navigation menu.
>
> Chip
>
> _______________________________________________
> 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/20100902/b0e8cf64/attachment-0001.htm>