[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