Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#29452 closed defect (bug) (fixed)

Install screen has no RTL

Reported by: ramiy Owned by: SergeyBiryukov
Priority: normal Milestone: 4.0
Component: Upgrade/Install Version: 4.0
Severity: normal Keywords: has-patch commit dev-reviewed
Cc: Focuses: rtl

Description

The new install screen, after selecting an RTL language (in my case Hebrew), has no RTL support.

Attachments (8)

install-rtl-test-1.png (63.8 KB ) - added by ramiy 12 years ago.
install-rtl-test-2.png (57.7 KB ) - added by ramiy 12 years ago.
install-rtl-test-3.png (42.1 KB ) - added by ramiy 12 years ago.
install-rtl-test-4.png (63.8 KB ) - added by ramiy 12 years ago.
29452.patch (1.5 KB ) - added by ocean90 12 years ago.
29452.diff (3.7 KB ) - added by johnbillion 12 years ago.
29452.2.diff (3.8 KB ) - added by johnbillion 12 years ago.
29452.2.patch (1.5 KB ) - added by pento 12 years ago.

Download all attachments as: .zip

Change History (21)

#1 @SergeyBiryukov
12 years ago

  • Component GeneralUpgrade/Install
  • Milestone Awaiting Review4.0

#2 @johnbillion
12 years ago

Argh, there's no RTL flag on the languages returned by the Translations API. If there was, we could switch the text direction on this screen in the same way we switch the button text.

This ticket was mentioned in IRC in #wordpress-dev by johnbillion. View the logs.


12 years ago

#4 @johnbillion
12 years ago

Patch coming up.

@ocean90
12 years ago

#5 @ocean90
12 years ago

29452.patch re-inits $GLOBALS['wp_locale'] after the default textdomain has changed.

Note: This only works in /build.

Last edited 12 years ago by ocean90 (previous) (diff)

@johnbillion
12 years ago

#6 @johnbillion
12 years ago

  • Keywords has-patch added

Dominik's patch was simpler than mine, so I've just extended that. 29452.diff also displays the WP_Locale::rtl_src_admin_notice() message on the install and setup-config screens when you're running src.

@johnbillion
12 years ago

#7 @johnbillion
12 years ago

29452.diff forgot the version check. Let's try 29452.2.diff.

Running src:

https://i.imgur.com/nWS4vjb.png

Not running src (or fudging your version number):

https://i.imgur.com/h9YQ2mc.png

#8 @nacin
12 years ago

  • Keywords commit dev-reviewed added

I'm fine with just 29452.patch for 4.0.

#9 @SergeyBiryukov
12 years ago

There's a typo in 29452.patch in line 207, should be $GLOBALS.

Last edited 12 years ago by SergeyBiryukov (previous) (diff)

@pento
12 years ago

This ticket was mentioned in IRC in #wordpress-dev by nacin. View the logs.


12 years ago

#12 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status newclosed

In 29669:

Re-initialize $wp_locale global after changing the default textdomain on installation screens.

props ocean90.
fixes #29452.

#13 @SergeyBiryukov
12 years ago

Found a related issue while testing this: #29464.

Note: See TracTickets for help on using tickets.