Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
10 Sep 2013 at 13:22 UTC
Updated:
29 Jul 2014 at 22:54 UTC
#2015789: Remove language_css_alter() (RTL stylesheets) in favor of HTML 'dir' attribute moved Drupal to a more modern way of handling RTL CSS.
However, not all "-rtl.css" references were removed.
This patch addresses the remainder.
| Comment | File | Size | Author |
|---|---|---|---|
| remove_remaining_rtl_css_references.patch | 3.88 KB | wim leers |
Comments
Comment #1
fubhy commentedDid you grep for any other leftover -rtl occurrences? If so, then I guess we can RTBC this already.
Comment #2
gábor hojtsyAgreed, looks good.
Comment #3
wim leers#1: I did.
Comment #4
joelpittetDo any of these need to be incorporated in the patch?
Searching 5583 files for "-rtl"
Searched for
-rtlexcludingvendor/*after applying the patch from the issue summary.Comment #5
gábor hojtsyNo this issue is about removing references / uses of the old -rtl.css autodiscovery feature. Image replacements for RTL still apply.
Comment #6
wim leersIndeed — but thanks for the double-checking! :)
Comment #7
joelpittetOk cool, RTBC +1:)
Comment #8
webchickCommitted and pushed to 8.x. Thanks!
Comment #9
gábor hojtsyYay, thanks all!
Comment #11
gábor hojtsySuperb, thanks!