Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
#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 CreditAttribution: 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
-rtl
excludingvendor/*
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!