[wp-trac] [WordPress Trac] #15863: Child Themes don't support RTL

WordPress Trac wp-trac at lists.automattic.com
Sun Jan 9 13:16:08 UTC 2011


#15863: Child Themes don't support RTL
-------------------------+----------------------
 Reporter:  ramiy        |       Owner:  yoavf
     Type:  enhancement  |      Status:  closed
 Priority:  normal       |   Milestone:
Component:  Template     |     Version:  3.0
 Severity:  critical     |  Resolution:  invalid
 Keywords:               |
-------------------------+----------------------
Changes (by ocean90):

 * keywords:  has-patch tested =>
 * status:  reopened => closed
 * resolution:   => invalid


Comment:

 Replying to [comment:11 ramiy]:
 > Why not add this to wordpress core?

 Because it could break more then it helps. It's the same reason, why
 ''style.css'' won't be loaded.
 Example: You have a ''#main'' part in the parent theme which needs some
 ''float:left'', in RTL CSS it will be ''float:right''. Now you have a
 child theme with a ''#main'' part too, but it doesn't need the ''float''
 anymore. And now you want to load the RTL CSS; you see, it will break the
 layout, because of the remaining ''float:right''.

 Two lead devs said the same:

 Quote by [comment:7 dd32]:
 > Iif we included the style.css, then including rtl.css might make sense.
 As we're not, then it'll potentially break themes for RTL rather than
 "fixing" them.

 Quote by [comment:6 yoavf]:
 > A child theme can be style-independent from the parent, and only re-use
 the php. In a case like this, forcing the rtl.css file to load will
 actually "break" the child theme.
 [[br]]

 There are now two possiblities to *fix* it by yourself, first is to import
 the CSS file via ''@import'' or use the plugin by SergeyBiryukov  (or add
 the function to your ''functions.php'').

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


More information about the wp-trac mailing list