Opened 12 years ago
Closed 12 years ago
#29464 closed defect (bug) (fixed)
setup-config.php should use RTL CSS files
| Reported by: | SergeyBiryukov | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.0 |
| Component: | Upgrade/Install | Version: | |
| Severity: | normal | Keywords: | has-patch commit dev-reviewed |
| Cc: | Focuses: | ui, rtl |
Description
Background: #29452
For RTL, we have install-rtl.css and buttons-rtl.css files. We use them in install.php, but not in setup-config.php. This leads to misplaced list, fields, and button (see the screenshots).
This is not a regression, but seems trivial to fix. We use wp_admin_css() in install.php, and it works fine in setup-config.php too.
Attachments (5)
Change History (10)
#2
@
12 years ago
Ah yes, we can do this now that we're bootstrapping WordPress. No side effects noticed? It should be fine, as the only difference between setup-config.php and install.php now is just one has a DB connection and one doesn't.
#3
@
12 years ago
Didn't see any side effects with 29464.patch.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
buttons.cssis already a dependency ofinstall.css.