[wp-trac] [WordPress Trac] #24450: Weak CSS selectors for WP Editor styles

WordPress Trac noreply at wordpress.org
Thu May 30 22:18:54 UTC 2013


#24450: Weak CSS selectors for WP Editor styles
--------------------------+--------------------
 Reporter:  ericlewis     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.6
Component:  Editor        |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+--------------------

Comment (by azaozz):

 Replying to [comment:2 ericlewis]:
 > It's tricky here, because the nature of the WP Editor is that it can be
 called multiple times per page, meaning if we were going to style it with
 IDs we would have to drop styles in an internal stylesheet, like the
 styling of the WP admin bar or the Twenty-somethings' custom headers. Or
 perhaps use !important rule for the wp_themeSkin declarations.

 Yeah, and both options (a body stylesheet generated from PHP or
 !important) don't sound that good.

 Agree with Sergey that `.wp_themeSkin tr.mceFirst .mceToolbar tr td` is
 quite specific. We could probably add `.mceEditor` in front of every
 `.wp_themeSkin` in [http://core.trac.wordpress.org/browser/trunk/wp-
 includes/css/editor.css editor.css] (about half of the selectors there)
 but it will always be possible to overwrite that with a selector
 containing IDs. Better option imho is to avoid using IDs when defining
 basic styles, or if not possible, at least to make them more specific:
 `#page > table tr td`, etc.

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


More information about the wp-trac mailing list