Opened 20 months ago
Closed 19 months ago
#18866 closed defect (bug) (duplicate)
setup-config.php styling is broken
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | UI | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | olleicua@…, kpayne@… |
Description
Expected (3.2) - http://cl.ly/0u120s3f2V162v0F1105
Actual (trunk) - http://cl.ly/1v2T3w1o331l1K3k0G0e
Product of the CSS merge, I imagine.
Attachments (3)
Change History (13)
comment:1
SergeyBiryukov — 20 months ago
Dirp.. I just read 17975 and realized the problem with my patch here's a better one.
My patch does include a css reference to an image in the wp-admin directory. This could be an issue in the cases that 17975 was worried about but in those cases it will simply display the button with no gradient and in most cases it will do the right thing. I don't see a better way around this.
Maybe we could replace this with a CSS3 gradient. We've done that in a lot of places.
I hesitate wanting to duplicate all of this CSS, but we won't have a choice.
Are there no browsers that don't yet support css gradients that we support? Also, is there a list somewhere of browsers we support?
Peace
~Sam
- Keywords has-patch added
Maybe we could replace this with a CSS3 gradient. We've done that in a lot of places.
But not all, see #16461. Maybe something for 3.4.
comment:8
SergeyBiryukov — 19 months ago
Related: #18576
- Cc kpayne@… added
gradient-add.diff test results:
- FF8 - passed
- Chrome 15 - passed
- IE7 - square button, no gradient
- IE8 - square button, no gradient
- IE9 - no gradient
- Opera - passed
- Safari - passed
The patch's comments hint that gradients won't work on IE < 10.
18866_fix_styles.patch test results:
- FF8 - passed
- Chrome 15 - passed
- IE7 - square button
- IE8 - square button
- IE9 - passed
- Opera - passed
- Safari - passed
18866.include_stylesheet.patch test results:
- stale, won't patch cleanly
comment:10
ocean90 — 19 months ago
- Milestone 3.3 deleted
- Resolution set to duplicate
- Status changed from new to closed
It's not directly setup-config.php. More _default_wp_die_handler, called by wp_die in wp-load.php.
See #17975.

Actually a product of #17975, see comment:ticket:17975:10.