Make WordPress Core

Opened 17 years ago

Closed 16 years ago

#6296 closed defect (bug) (fixed)

RTL CSS for 2.5

Reported by: nbachiyski's profile nbachiyski Owned by:
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: Cc:

Description

Prepared by Mani Monajjemi.

Attachments (7)

rtl-for-25.diff (8.3 KB) - added by nbachiyski 17 years ago.
wp-admin-rtl.zip (7.7 KB) - added by nbachiyski 17 years ago.
The whole zip will be more convinient.
wp-admin-rtl-edit2.patch (3.3 KB) - added by mani_monaj 17 years ago.
New Improvements
colors-rtl-fix.patch (467 bytes) - added by mani_monaj 17 years ago.
Colors RTL fix
wp-admin-rtl-edit3.patch (6.4 KB) - added by mani_monaj 17 years ago.
Improvements
wp-admin-rtl-edit4.patch (1.5 KB) - added by mani_monaj 17 years ago.
kubrick-rtl.css (1.7 KB) - added by mani_monaj 17 years ago.
New Kubrik's rtl.css

Download all attachments as: .zip

Change History (28)

@nbachiyski
17 years ago

The whole zip will be more convinient.

#1 @ryan
17 years ago

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

(In [7398]) RTL CSS from Mani. fixes #6296

@mani_monaj
17 years ago

New Improvements

#2 @mani_monaj
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

I added a patch. It improves some bugs on IE & Upload Page.

#3 @ryan
17 years ago

(In [7476]) RTL CSS from mani_monaj. see #6296

#4 follow-up: @ryan
17 years ago

I'm not sure if the colors rtl css is being loaded correctly. I think wp_admin_css_uri() needs to be fixed.

#5 in reply to: ↑ 4 @mani_monaj
17 years ago

Replying to ryan:

I'm not sure if the colors rtl css is being loaded correctly. I think wp_admin_css_uri() needs to be fixed.

I've just fixed it, I attached the patch.

@mani_monaj
17 years ago

Colors RTL fix

#6 @ryan
17 years ago

(In [7477]) Fix loading of colors RTL css. Props mani_monaj. see #6296

#7 @mani_monaj
17 years ago

Could you please ignore wp-admin-rtl-edit3.patch and let me fix all issues and submit the final patch instead.

#8 @ryan
17 years ago

see #6365 for some more RTL work

#9 follow-up: @ryan
17 years ago

Mani, take a look at #6365 and see if you can incorporate those changes.

#10 in reply to: ↑ 9 @mani_monaj
17 years ago

Replying to ryan:

Mani, take a look at #6365 and see if you can incorporate those changes.

Sure, I'll take a look and try to merge the changes and prepare the final patch.

#11 @mani_monaj
17 years ago

I manually merged our new patches with proposed changes in #6365 and attached the file. The only issue is the default font for RTL which has been leaved as Tahoma.

The RTL now looks good both on Firefox and IE7.

The remaining issues in RTL mode are:

  1. The RSS/Text widgets in widget page; When opening the widget edit panel:

In Firefox: The box becomes bigger than standard column size.
In IE7: The box disappears!

  1. In pages with lists and delete button (e.g. manage links), when hovering the delete button, the whole table moves some pixels to left

@mani_monaj
17 years ago

Improvements

#12 @ryan
17 years ago

(In [7494]) RTL fixes from mani_monaj and RanYanivHartstein. see #6296 #6365

#13 @mani_monaj
17 years ago

@Ryan

The file theme-editor-rtl.css is not in changeset [7494]. Were there any problems with that file?

#14 @ryan
17 years ago

(In [7501]) RTL fixes from mani_monaj and RanYanivHartstein?. see #6296 #6365

#15 @ryan
17 years ago

I forgot to svn add theme-editor-rtl.css.

#16 @thee17
17 years ago

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

#17 @mani_monaj
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

As some original CSS files changed, I attached a new patch.

#18 @ryan
17 years ago

(In [7552]) RTL fixes from mani_monaj. see #6296

#19 @mani_monaj
17 years ago

We wrote a new rtl.css file for Kubrik, I attach this file too, it is completely different from the kubrik's original rtl.css file.

@mani_monaj
17 years ago

New Kubrik's rtl.css

#20 @ryan
17 years ago

(In [7553]) Updated RTL for default theme from mani_monaj. see #6296

#21 @ryan
16 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.