[wp-trac] Re: [WordPress Trac] #6380: the_excerpt & [gallery]
WordPress Trac
wp-trac at lists.automattic.com
Tue Apr 1 03:35:34 GMT 2008
#6380: the_excerpt & [gallery]
----------------------------+-----------------------------------------------
Reporter: DD32 | Owner: anonymous
Type: defect | Status: new
Priority: normal | Milestone: 2.5.1
Component: Administration | Version: 2.5
Severity: normal | Resolution:
Keywords: has-patch |
----------------------------+-----------------------------------------------
Comment (by tellyworth):
One issue that's related to the stylesheet location is that in 2.5.0, you
can't have multiple galleries on one page with different numbers of
columns. It looks like Andy's patch will fix this.
I'd advocate serving up a static external stylesheet rather than making it
dynamic. Though it would initially have only the gallery styles in it,
the static stylesheet could eventually contain default styles for other
built-in shortcodes and content-related things.
Seems to me that the cost of scanning forward for {{{[gallery]}}} tags and
conditionally including an inline stylesheet is out of proportion to the
(minor) benefit of not serving up the gallery stylesheet on pages that
don't use it.
As for the_excerpt: what about a toggle or wrapper function that would
temporarily disable shortcode parsing and just return the literal
"[gallery]" instead?
--
Ticket URL: <http://trac.wordpress.org/ticket/6380#comment:4>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list