[wp-trac] [WordPress Trac] #31332: RTL CSS generation: Switch from CSSJanus to RTLCSS

WordPress Trac noreply at wordpress.org
Sat Feb 14 20:42:11 UTC 2015


#31332: RTL CSS generation: Switch from CSSJanus to RTLCSS
--------------------------------------------------+------------------
 Reporter:  ocean90                               |       Owner:
     Type:  defect (bug)                          |      Status:  new
 Priority:  normal                                |   Milestone:  4.2
Component:  Build/Test Tools                      |     Version:
 Severity:  normal                                |  Resolution:
 Keywords:  has-patch dev-feedback needs-testing  |     Focuses:
--------------------------------------------------+------------------

Comment (by ocean90):

 [attachment:build.diff] is a diff from the CSS files.
 (Note: The change in `wp-admin/css/ie-rtl.css` is correct and fixes a bug
 in cssjanus.)
 [[BR]]

 > Would it possible to get some sample vizrecs for before (w/ CSSJanus)
 and after (w/ RTLCSS)?
 There isn't much difference, but I just noticed that the shadow of the
 Press This button is now switched in RTL, see
 https://cloudup.com/c6tuCzztRqA.
 [[BR]]

 > The one thing I think I would do differently (and my above patch does
 it) is to keep cssjanus as a command,
 I don't think this is really necessary. We already have a custom RTL task
 (`grunt rtl`) which should be used.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/31332#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list