Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#46868 closed defect (bug) (fixed)

Site Health : RTL css doesn't load

Reported by: mohadeseghasemi's profile mohadeseghasemi Owned by: desrosj's profile desrosj
Milestone: 5.2 Priority: normal
Severity: normal Version: 5.2
Component: Site Health Keywords: has-screenshots site-health has-patch
Focuses: rtl, administration Cc:

Description

I check the Site Health Status in RTL version, it seems the RTL css file doesn't load and t loads the default styles.

Attachments (3)

Site Health Status.png (34.8 KB) - added by mohadeseghasemi 6 years ago.
RTL-view.png (18.4 KB) - added by ebrahimzadeh 6 years ago.
46868.patch (344 bytes) - added by ebrahimzadeh 6 years ago.

Download all attachments as: .zip

Change History (10)

#1 @ebrahimzadeh
6 years ago

  • Severity changed from major to normal

Hi @mohadeseghasemi

Same result here. For test I rename site-health-rtl.min to site-health.min, it works, see the attachment.

@ebrahimzadeh
6 years ago

#2 @afercia
6 years ago

  • Keywords site-health added
  • Milestone changed from Awaiting Review to 5.2
  • Version set to trunk

Can reproduce. At the very least, the site-health CSS needs to be added to the $rtl_styles array in script-loader.php.

@ebrahimzadeh
6 years ago

#3 @ebrahimzadeh
6 years ago

Please check the patch.

#4 @ebrahimzadeh
6 years ago

  • Keywords has-patch added; needs-patch removed

#5 @desrosj
6 years ago

  • Owner set to desrosj
  • Status changed from new to reviewing

#6 @desrosj
6 years ago

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

In 45168:

Site Health: Ensure Site Health pages are correctly displayed for RTL locales.

Props: ebrahimzadeh, afercia, mohadeseghasemi.
Fixes #46868.

#7 @spacedmonkey
6 years ago

  • Component changed from Administration to Site Health
Note: See TracTickets for help on using tickets.