[wp-trac] [WordPress Trac] #9015: New theme for 3.0 -- Twenty Ten
WordPress Trac
wp-trac at lists.automattic.com
Mon Mar 1 14:31:37 UTC 2010
#9015: New theme for 3.0 -- Twenty Ten
----------------------------+-----------------------------------------------
Reporter: dd32 | Owner:
Type: task (blessed) | Status: new
Priority: normal | Milestone: 3.0
Component: Themes | Version:
Severity: normal | Keywords:
----------------------------+-----------------------------------------------
Comment(by dougal):
Replying to [comment:140 Viper007Bond]:
> Indeed, it broke my lightbox plugin. I had to recode it to use a random
number instead of the gallery's ID.
>
> However really it should be a class rather than an ID incase you include
a gallery from another post into a post or make the gallery show up twice.
Probably a discussion for a separate ticket though.
Not sure why (or how) you'd include the same gallery more than once in the
same view... Maybe if you had a widget that expanded a gallery shortcode
on the same view as a post with the same gallery. But I think that would
be a pretty rare edge case.
But even if we moved the gallery-id into class, the main problem still
stands: the function in question should only remove the inline `style`
element, and leave the `div` declaration intact.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/9015#comment:142>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list