Front Page with 2 galleries doesn't respect settings for each

[MtDewVirus]

If your front page has two posts with galleries and each gallery is set to display a different number of columns, the CSS from the oldest (lowest displayed post on the page) is used.

Publish a post with a gallery and set columns="2" and then publish another post without setting the columns. The CSS for each gallery is in the HTML, but both posts will use the CSS to display 2 columns.

Tested on Firefox 3.0.1 Mac and r8797.

[DD32]

Media is getting an overhaul in 2.8, This'll probably be skiped for the 2.7 release.

[jamescollins]

This is also a problem on a WP page that contains multiple [gallery] shortcodes with different columns=x parameters.

The CSS rules from the last gallery override all the other galleries.

This problem is also discussed <a href="​http://en.forums.wordpress.com/topic/problem-with-gallery-columns">in the WP.com forums</a>.

[jamescollins]

patch against r11091

[jamescollins]

Simple patch (7678.diff) adds a unique id to each gallery (based on the post id).

The CSS rules ouptutted with each gallery are then specific to that gallery id, and shouldn't affect any other galleries on the same page.

[DD32]

@jamescollins: Nice idea, And its backwards compatible too!

[jamescollins]

Replying to DD32:

@jamescollins: Nice idea, And its backwards compatible too!

Thanks DD32.

The only problem I can think of would be that if you have the multiple galleries for the same page/post id on a single page.

For example, you have page (id 5) which has a lot of text, with two gallery shortcodes on it:

[gallery columns=1]

text here.

[gallery columns=4]

In this case, the CSS class for both galleries would be set to galleryid-5, so the same problem would still occur (both galleries would be displayed with 4 columns)

The other option could be to use a static counter variable that is incremented by one every time the [gallery] shortcode is parsed. This guarantees a unique class selector for each gallery on each page.

[jamescollins]

Alternative patch using static

[jamescollins]

Replying to jamescollins:

The other option could be to use a static counter variable that is incremented by one every time the [gallery] shortcode is parsed. This guarantees a unique class selector for each gallery on each page.

Patch for this attached (see 7678_alternative.diff).

Which solution do you think is better?

[DD32]

Static var is better, Mind you, Its rare to have 2 galleries with the same postID visible at once.. (esp since it doesnt work right now)

[jamescollins]

Replying to DD32:

Static var is better, Mind you, Its rare to have 2 galleries with the same postID visible at once.. (esp since it doesnt work right now)

The first solution semantically cleaner, but the static var solution is more bullet proof.

What are the chances of getting this fixed for 2.8?

Thanks.

[DD32]

The first solution semantically cleaner, but the static var solution is more bullet proof.

Indeed.

Theres no real reason why we cant insert both classes into the HTML, But style based off the static version.. :)

Some dev feedback on commit or punt would be good.

[jamescollins]

Replying to DD32:

Theres no real reason why we cant insert both classes into the HTML, But style based off the static version.. :)

That's a good idea. Adding both the galleryid-x class and the gallery-y classes, but styling the gallery using the (more unique) gallery-y class sounds good to me.

[Denis-de-Bernardy]

more or less the same fix as #9400

[hakre]

I suggest to close #9400 (there) as a duplicate because the suggestion in #7678 (gere) are better.

[jamescollins]

Replying to DD32:

Theres no real reason why we cant insert both classes into the HTML, But style based off the static version.. :)

Some dev feedback on commit or punt would be good.

I've created 7678_combined.diff. It creates a unique gallery-x class (by using a static variable) which is used for styling the gallery, as well as the galleryid-y class which contains the post id of the gallery.

[jamescollins]

uses unique css id for styling, and a galleryid-x class

[Denis-de-Bernardy]

$id should be $post->ID, else you've the ID of the last attachment instead of that of the post. once that's updated, but +1 tested commit and please close #9400 as dup.

[jamescollins]

Replying to Denis-de-Bernardy:

$id should be $post->ID, else you've the ID of the last attachment instead of that of the post. once that's updated, but +1 tested commit and please close #9400 as dup.

Thanks Denis.

In 7678_combined2.diff I have changed the attachment loop to use $attid instead of $id, so it doesn't conflict with the $id gallery parameter.

[jamescollins]

Replying to Denis-de-Bernardy:

Thanks for testing Denis. Is there any chance of getting this included in WP 2.8?

Thanks.

[Denis-de-Bernardy]

I'm pretty sure Ryan will add it. He and Andrew are going through the commit list, and the patch is totally harmless.

[Denis-de-Bernardy]

see also #8181 or one of the many related tickets, which may be worth looking into at one point.

[azaozz]

(In [11339]) Support more than one gallery on the same page, props jamescollins, fixes #7678