WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#6121 closed defect (bug) (fixed)

RTL CSS in Login

Reported by: Ali_ix Owned by:
Milestone: 2.5 Priority: normal
Severity: normal Version: 2.5
Component: Administration Keywords: rtl login css has-patch
Focuses: Cc:

Description

Please include the 'login-rtl.css' file in login page after 'login.css' file.
And please remove the 'dir="rtl"' property on html tag there.

Attachments (1)

login-rtl-css.diff (562 bytes) - added by nbachiyski 6 years ago.

Download all attachments as: .zip

Change History (4)

nbachiyski6 years ago

comment:1 nbachiyski6 years ago

  • Keywords has-patch added
  • Milestone changed from 2.6 to 2.5

Attached is a patch, which loads login-rtl.css if we have the proper $text_direction.

Why shall we remove dir="rtl"?

comment:2 ryan6 years ago

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

(In [7203]) Include login rtl css. Props nbachiyski. fixes #6121

comment:3 Ali_ix6 years ago

Thanks for the fix.

Since we can handle the page direction in CSS, it would be unnecessary (dir="rtl"). By the way, it isn't an issue and it works.

Note: See TracTickets for help on using tickets.