cleaner-gallery icon indicating copy to clipboard operation
cleaner-gallery copied to clipboard

Use of `wp-caption-text` class on captions

Open NateWr opened this issue 9 years ago • 2 comments
trafficstars

Hi Justin, I've been looking at using the WP Featherlight plugin in conjuction with Cleaner Gallery.

WP Featherlight selects the caption for display with the wp-caption-text class. Cleaner Gallery doesn't include this class. It only uses the gallery-caption class and it's not filterable.

Core uses both and re-uses wp-caption-text for it's caption shortcode, which I did not know existed until just now.

I'll give Rob Nue (behind WP Featherlight) a heads up about this as well. But on looking into this I think it's appropriate for the <figcaption> to have the wp-caption-text class.

I wondered what you thought about adding this class or just making the caption class filterable like the image classes.

NateWr avatar Feb 06 '16 16:02 NateWr

I was against core adding this class to galleries. It actually broke quite a number of theme designs when they added it (previously, it was only added to the caption shortcode). So, I almost consider it a feature at this point.

Given how long its been, it might be safe to make the change. I'll want to test it with my themes though.

justintadlock avatar Feb 06 '16 17:02 justintadlock

Thanks Justin, that makes sense and I can see the conflict. In this case, it was fairly easy to strip out the key features of Cleaner Gallery for use in my theme, so it's not a critical issue for me. The improved markup in core is a lot better than the old stuff when this was a lot more necessary. :)

NateWr avatar Feb 08 '16 11:02 NateWr