Widgets Panel needs to be RTLized

[rasheed]

Snapshot: ​http://img577.imageshack.us/img577/9193/1212t.gif

Location: wp-admin/widgets.php (Wordpres-ar/rtl)

#widget-list .widget {

border: 0 none;
float: left; (float should be right)
margin: 0 10px 15px;
width: 250px;

}

--
Edit by yoavf: edited title to reflect new patch that RTLize the whole widgets panel

This ticket made me realized Widgets UI has never been proprely RTLized.

Attached files fix that, finally

Let's do just the float for 3.1 and hit the rest in 3.2. Touching the widgets JS is scary at this stage.

(In [17216]) Quick RTL CSS fix in widgets admin. see #16082, props SergeyBiryukov.

Refreshed patch for trunk. Would be great if we can get this in 3.2.

Another refresh

Replying to nacin:

Let's do just the float for 3.1 and hit the rest in 3.2. Touching the widgets JS is scary at this stage.

I agree. Touching the JS at this stage can cause problems, and it isn't worth the risk.

Replying to maor:

I agree. Touching the JS at this stage can cause problems, and it isn't worth the risk.

Have you tested the patch? It's pretty straightforward.
This has already been punted from 3.1, and should really go in 3.2.

That was said after we were in RC in 3.1. The JS seems pretty basic ultimately.

The full patch is good but do we really want to switch over the widgets screen about year and a half after is was released? Would probably seem strange to all RTL users.

Replying to azaozz:

The full patch is good but do we really want to switch over the widgets screen about year and a half after is was released? Would probably seem strange to all RTL users.

That's a valid concern, but I think that in this case it's better late than never - it really doesn't make sense the way it currently is.

Replying to azaozz:

The full patch is good but do we really want to switch over the widgets screen about year and a half after is was released? Would probably seem strange to all RTL users.

I think the current widget's structure in WordPress RTL is fine.

azaozz please can you review this and either commit for 3.2 or punt with an explanation why.

Think this needs some UX feedback before we can commit. See the last comments by @yoavf and @rasheed.

Just to reiterate - though it currently works, it is reversed, and doesn't match the rest of the UI.
It doesn't make sense to keep it the way it is just because it is fine. We should strive for perfection :)

I'm with yoav. We designed the screen with a hierarchy/flow, and better late than never for it to make sense in RTL.

In [18095]:

Widgets RTL fixes, props yoavf, fixes #16082