Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#15984 closed defect (bug) (fixed)

RTL For the theme Installer

Reported by: yoavf Owned by: yoavf
Priority: normal Milestone: 3.1
Component: RTL Version: 3.1
Severity: normal Keywords: has-patch
Cc:

Description

Theme installer is missing RTL css.

Attached patch adds the missing bits and also removes hard-coded styles in wp-admin/includes/theme-install.php and wp-admin/themes.php

Attachments (1)

theme-install.rtl.patch​ (8.5 KB) - added by yoavf 2 years ago.

Download all attachments as: .zip

Change History (4)

yoavf — 2 years ago

comment:1   nacin — 2 years ago

Sweet, thanks.

In the future, no need to patch the compressed scripts -- the patches end up stale if anything unrelated changes. Probably not exactly a big deal for the small files you typically deal with though.

(Off topic for the ticket, can you also glance at #15974?)

comment:2   nacin — 2 years ago

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

(In [17149]) RTL for theme installer. props yoavf, fixes #15984.

comment:3   nacin — 2 years ago

(In [17150]) Add files missing in [17149]. see #15984.

Note: See TracTickets for help on using tickets.