[wp-trac] [WordPress Trac] #18497: More flexible overriding of gallery short code default CSS

WordPress Trac wp-trac at lists.automattic.com
Wed Aug 24 18:10:14 UTC 2011


#18497: More flexible overriding of gallery short code default CSS
-------------------------+------------------------------
 Reporter:  jb510        |       Owner:  jb510
     Type:  enhancement  |      Status:  closed
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Media        |     Version:  3.2.1
 Severity:  minor        |  Resolution:  wontfix
 Keywords:  has-patch    |
-------------------------+------------------------------
Changes (by jb510):

 * status:  assigned => closed
 * resolution:   => wontfix


Comment:

 Part of my thought behind this patch was that a lot of "average" users can
 add CSS to their theme but don't have the skills to add a filter in PHP
 further that in situations like muiltsite (where we run WordPress.com
 Custom CSS plugin) users can sometimes add CSS but aren't allowed to add
 PHP.  However I still see the utility in auto-generating the CSS such that
 size/columns/etc can all be accounted for dynamically.

 Gary's right that I don't see a huge percentage of people needing the
 ability to disable the default CSS on a gallery by gallery basis and...
 now for the embarrassing admission... I'd forgotten the specificity needed
 could be added with body classes, I was thinking overriding inline css
 required !important.

 So while I personally still think as a general rule short codes that auto-
 generate inline css should have an attribute to disable that CSS, I
 completely understand if that philosophy conflicts with the philosophy of
 things done in core.

 TY both for looking at it.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/18497#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list