Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#20583 closed defect (bug) (fixed)

RTL style fixes for the install theme info sidebar

Reported by: ocean90's profile ocean90 Owned by: ryan's profile ryan
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.4
Component: RTL Keywords: has-patch
Focuses: Cc:

Description

See before.png and after.png.

Attachments (6)

20583.patch (1.6 KB) - added by ocean90 11 years ago.
before.png (94.9 KB) - added by ocean90 11 years ago.
after.png (93.4 KB) - added by ocean90 11 years ago.
20583.diff (1.1 KB) - added by ryan 11 years ago.
20673.use-rtl-star.patch (1.2 KB) - added by ocean90 11 years ago.
20583.2.diff (1.1 KB) - added by helenyhou 11 years ago.

Download all attachments as: .zip

Change History (15)

@ocean90
11 years ago

@ocean90
11 years ago

@ocean90
11 years ago

#1 follow-up: @helenyhou
11 years ago

Looks good. There's an issue with the sidebar collapse but it's not theme install specific.

#2 in reply to: ↑ 1 @ocean90
11 years ago

Replying to helenyhou:

Right, this is fixed in #20585.

#3 @ryan
11 years ago

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

In [20673]:

RTL style fixes for the install theme info sidebar. Props ocean90. fixes #20583

#4 @helenyhou
11 years ago

  • Keywords needs-patch added; has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

There's a stars.rtl.png now, so ocean90's small hack can be removed.

@ryan
11 years ago

#5 follow-up: @ryan
11 years ago

Like so?

#6 @ocean90
11 years ago

  • Keywords has-patch added; needs-patch removed

#7 in reply to: ↑ 5 @ocean90
11 years ago

Replying to ryan:

Like so?

No, more like 20673.use-rtl-star.patch.

@helenyhou
11 years ago

#8 @helenyhou
11 years ago

20583.2.diff gets rids of some unnecessary old star CSS from the RTL sheet and moves the new star CSS into the more appropriate spot, since it also affects the details in the list table. So maybe more like that.

#9 @ryan
11 years ago

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

In [20763]:

Use stars-rtl.png for RTL start support in the theme installer. Props helenyhou, ocean90. fixes #20583

Note: See TracTickets for help on using tickets.