[theme-reviewers] Theme requires slider images to be of a specific size.

Josh Pollock jpollock412 at gmail.com
Thu Mar 14 22:50:53 UTC 2013


Thanks chip, I will include this recommendation in the ticket.


On Thu, Mar 14, 2013 at 6:16 PM, Chip Bennett <chip at chipbennett.net> wrote:

> I would say, then, that the problem is the *implementation*, not the image
> size. The image slider needs to be implemented by defining a custom image
> size, via add_image_size(), and then query those images using, e.g.
> the_post_permalink( $size ).
>
>
> On Thu, Mar 14, 2013 at 4:39 PM, Josh Pollock <jpollock412 at gmail.com>wrote:
>
>> Chip-
>> It is not using add_image_size() . I tested the slider with two random
>> sized images. It displayed the first one, but it didn't work as a slider
>> and the second one didn't show up at all. I switched the images to ones
>> with the specified dimensions, but nothing changed. Every time I try and
>> save options I get a Warning: Cannot modify header information - headers
>> already sent error, so we've got a fail here, I'm just trying to figure out
>> exactly what to require.
>>
>>
>> On Thu, Mar 14, 2013 at 4:13 PM, Chip Bennett <chip at chipbennett.net>wrote:
>>
>>> Is the Theme using add_image_size() to define that custom image size?
>>>
>>> If so, that is normal, acceptable content presentation. (Though noting
>>> any custom image sizes in the Theme documentation is usually recommended,
>>> especially for large images such as that one.)
>>>
>>> Chip
>>>
>>>
>>> On Thu, Mar 14, 2013 at 4:06 PM, Josh Pollock <jpollock412 at gmail.com>wrote:
>>>
>>>> I am reviewing a theme that has a built in slider that requires images
>>>> to be exactly 963px x 350px. Is this acceptable per the guidelines or does
>>>> dictating the presentation of content in an unacceptable way.
>>>>
>>>> _______________________________________________
>>>> 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/20130314/d37797a0/attachment.htm>


More information about the theme-reviewers mailing list