Allow choosing multiple header images

[wikicms]

Please, see small screenshot :)

Description: This is the theme TwentyEleven. I want choose multiple header images, (for example 4,5,6), but i cannot. Only random or only one. Thanks.

[ericlewis]

I'd say this falls under either child-theme or plugin development.

[jphase]

Allow multiple header images

[jphase]

Hey everyone, this is my first diff so if I did anything wrong (syntactically or otherwise) then please give me feedback. I want to learn the right way to contribute :)

I updated the UI to have checkboxes next to each image and a bit of jQuery to toggle between the "Random" radio and the selected checkboxes. The "Random" radio still just pulls random images from the default images and I modified get_random_header_image() to allow an optional paths array to be passed which will allow a random index to be returned.

Perhaps we could eventually allow an interface to save a preferred order of the selected images later?

[jphase]

Updated diff to fix minor syntax discrepancies

[simonwheatley]

Hi Jphase,

I don't have time to test your patch right now (bedtime!), sorry, but I can point you towards the existing header tests. If you have the new style WordPress development repo, they are in tests/phpunit/tests/image/header.php, or here's a link to them in SVN: ​http://develop.svn.wordpress.org/trunk/tests/phpunit/tests/image/header.php

Here's some info on automated tests, in case it's useful: ​http://make.wordpress.org/core/handbook/automated-testing/

(Note the stuff on @ticket, that tripped me up when I couldn't work out why my tests weren't working.)

Hope this helps.

[jphase]

Replying to simonwheatley:

Hi Jphase,

I don't have time to test your patch right now (bedtime!), sorry, but I can point you towards the existing header tests. If you have the new style WordPress development repo, they are in tests/phpunit/tests/image/header.php, or here's a link to them in SVN: ​http://develop.svn.wordpress.org/trunk/tests/phpunit/tests/image/header.php

Here's some info on automated tests, in case it's useful: ​http://make.wordpress.org/core/handbook/automated-testing/

(Note the stuff on @ticket, that tripped me up when I couldn't work out why my tests weren't working.)

Hope this helps.

Hey Simon,

Thanks for the info; it was exactly what I needed. I ran the phpunit test from the "header" group and it appears like things are okay. Here's a screenshot from my terminal. Let me know if I did this right or if I missed anything.

[simonwheatley]

I'd also try running the tests with the ticket number in the group parameter too, this will ensure any tests relating to this ticket are not skipped:

phpunit --group header,18959

[jphase]

Gotcha, thx for all the info Simon. I'm sure nothing is different on this test since there aren't any actual tests attached to this ticket but just to be thorough, here's the latest output:

[celloexpressions]

The headers admin page is now only accessible to no-js users and a couple of other edge cases where the Customizer isn't supported. Headers in the Customizer could potentially benefit from this for default headers, although this functionality is present for user-uploaded ones of course.

Is anyone still interested in making improvements here, given that?

[melchoyce]

You can kind of already do this by removing the images you don't want to appear randomized. This seems like it would need a special UI for a very small use case, so I’m going to close.