[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