Followup from #2015789-73: Remove language_css_alter() (RTL stylesheets) in favor of HTML 'dir' attribute, it is preferred to quote the attribute selector. This patch changes all instances of [dir=rtl] to [dir="rtl"]

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jessebeach’s picture

Status: Needs review » Reviewed & tested by the community

Looks great. Good hustle echoz.

Wim Leers’s picture

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Needs a reroll

git ac https://drupal.org/files/quote-rtl-attribute.patch
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 80515  100 80515    0     0  16910      0  0:00:04  0:00:04 --:--:-- 17711
error: patch failed: core/themes/seven/install-page.css:50
error: core/themes/seven/install-page.css: patch does not apply
echoz’s picture

Status: Needs work » Needs review
FileSize
77.81 KB

commit all the things! :-D

penyaskito’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Avoid commit conflicts

Looks ok.

jessebeach’s picture

Looks great, let's get this in before it becomes invalid. It touches too much code to continue to apply for too long.

jessebeach’s picture

#4: quote-rtl-attribute-2030925-4.patch queued for re-testing.

YesCT’s picture

Issue tags: +RTBC July 1

This issue was RTBC and passing tests on July 1, the beginning of API freeze.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -Avoid commit conflicts

Committed b48f8fd and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.