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

Otto otto at ottodestruct.com
Thu Mar 14 22:37:57 UTC 2013


There is no the_post_permalink. I presume you meant
wp_get_attachment_image() or something similar?

-Otto


On Thu, Mar 14, 2013 at 5: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
>


More information about the theme-reviewers mailing list