Opened 4 years ago

Closed 4 years ago

#9735 closed defect (bug) (fixed)

rtl stylesheets for color scheme not loaded

Reported by: yoavf Owned by: anonymous
Priority: normal Milestone: 2.8
Component: Administration Version: 2.8
Severity: major Keywords: has-patch tested css rtl
Cc:

Description

When blog is set to RTL, color-fresh.css is called twice. color-fresh-rtl.css is not loaded at all.

admin-header.php output:

<link rel='stylesheet' id='colors-css'  href='http://localhost:8888/trunk/wp-admin/css/colors-fresh.css?ver=20090503' type='text/css' media='all' />
<link rel='stylesheet' id='colors-rtl-css'  href='http://localhost:8888/trunk/wp-admin/css/colors-fresh.css?ver=20090503' type='text/css' media='all' />

Attachments (1)

9735.patch​ (949 bytes) - added by hakre 4 years ago.
Fixed.

Download all attachments as: .zip

Change History (6)

comment:1   Denis-de-Bernardy — 4 years ago

  • Milestone changed from Unassigned to 2.8

comment:2   hakre — 4 years ago

Can reproduce. Will work on a patch now.

hakre — 4 years ago

Fixed.

comment:3   hakre — 4 years ago

  • Keywords has-patch tested css added; needs-patch, css, removed

someone has out the replace function before instead of after the filters. this is fixed now.

a incosistency i saw but just reported here FYI:

color handle has src (bool) true
color-rtl handle has src (int) 1

comment:4   yoavf — 4 years ago

Working, thanks hakre

comment:5   ryan — 4 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [11219]) Fix RTL color stylesheet loading. Props hakre. fixes #9735

Note: See TracTickets for help on using tickets.