Opened 15 years ago
Closed 15 years ago
#11502 closed defect (bug) (fixed)
Some buttons have gone square in 2.9
Reported by: | caesarsgrunt | Owned by: | azaozz |
---|---|---|---|
Milestone: | 2.9.1 | Priority: | normal |
Severity: | major | Version: | 2.9 |
Component: | UI | Keywords: | |
Focuses: | Cc: |
Description
After upgrading some of my live sites to 2.9, I see that some of the buttons have gone square with slightly rounded corners, as opposed to the capsule buttons which 2.8 had everywhere.
The change is not global, nor is there any apparent rule as to which buttons are square and which round - see attached screenshots.
The change did not appear at any point during the 2.9 beta and RC process - yesterday's nightly didn't even have it, so far as I am aware.
The buttons are now inconsistent throughout WP, and as a result the change is highly counter-intuitive and confusing.
Attachments (5)
Change History (30)
#1
@
15 years ago
Not even major action buttons (the blue ones) are one consistently the same shape - compare the publish buttons on QuickPress and New Post...
#2
@
15 years ago
Seems to be caused by a conflict between
.submit input, .button, .button-primary, .button-secondary, .button-highlighted, #postcustomstuff .submit input { -moz-border-radius-bottomleft:11px; -moz-border-radius-bottomright:11px; -moz-border-radius-topleft:11px; -moz-border-radius-topright:11px; ...
and
textarea, input[type="text"], input[type="password"], input[type="file"], input[type="button"], input[type="submit"], input[type="reset"], select { -moz-border-radius-bottomleft:4px; -moz-border-radius-bottomright:4px; -moz-border-radius-topleft:4px; -moz-border-radius-topright:4px; ...
on line 379 and line 1, respectively, of admin.dev.css
#3
@
15 years ago
Specifically, the latter snippet shouldn't apply to input[type="button"], input[type="submit"], input[type="reset"]
.
#5
follow-ups:
↓ 6
↓ 8
@
15 years ago
- Keywords has-patch added; needs-patch removed
Patch fixes the bug by removing the offending selectors.
(Side note : As it happens, I prefer the square buttons... but the shape should be consistent, and should not be changed without discussion.)
#6
in reply to:
↑ 5
@
15 years ago
Replying to caesarsgrunt:
Patch fixes the bug by removing the offending selectors.
(Side note : As it happens, I prefer the square buttons... but the shape should be consistent, and should not be changed without discussion.)
Tested the patch on Firefox 3.5 and Chrome latest.
I also like the square buttons more.
#8
in reply to:
↑ 5
;
follow-up:
↓ 10
@
15 years ago
- Owner set to azaozz
- Status changed from new to assigned
Replying to caesarsgrunt:
Patch fixes the bug by removing the offending selectors.
(Side note : As it happens, I prefer the square buttons... but the shape should be consistent, and should not be changed without discussion.)
Indeed I believe the change was introduced by accident and we should switch this back to the rounded buttons for 2.9.1
#10
in reply to:
↑ 8
@
15 years ago
Replying to westi:
Replying to caesarsgrunt:
Patch fixes the bug by removing the offending selectors.
(Side note : As it happens, I prefer the square buttons... but the shape should be consistent, and should not be changed without discussion.)
Indeed I believe the change was introduced by accident and we should switch this back to the rounded buttons for 2.9.1
You shure? The finer rounded-corner buttons were introduced for 2.9 weren't they? I thought this is one of the visual changes for the release.
#11
@
15 years ago
This was certainly introduced by[12448] / [12449], and I'm fairly sure it was unintentional since it isn't mentioned in the commit description and nor is a ticket linked to which describes the change. Also, the change is not universal; it just depends on the type of button within the html and not on what the button does.
#14
@
15 years ago
It wasn't intentional, sorry about that (some browsers really hate to update styles despite that all caches are disabled).
@caesarsgrunt removing these selectors would leave buttons without classes unstyled, better to make sure we apply the classes properly.
Can you confirm thunk looks as it should, before adding this to the 2.9 branch. Don't want to repeat the same mistake :)
#15
@
15 years ago
Working in trunk, but you busted the cache for globa.css instead of for wp-admin.css :)
#17
follow-ups:
↓ 18
↓ 23
@
15 years ago
- Keywords has-patch commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
Still getting square buttons in 2.9.1.
Yes, I've cleared my cache.
#18
in reply to:
↑ 17
;
follow-up:
↓ 19
@
15 years ago
Replying to caesarsgrunt:
Still getting square buttons in 2.9.1.
Yes, I've cleared my cache.
A little more details wouldn't hurt: browser, os, which admin page(s) etc.
#19
in reply to:
↑ 18
;
follow-up:
↓ 21
@
15 years ago
Replying to scribu:
A little more details wouldn't hurt: browser, os, which admin page(s) etc.
Browser: Firefox 3.5, Safari 4.0, Chrome 4.0...
OS: Doesn't make any difference, but if you're interested it's OS X 10.6.2.
Which pages: See the original ticket and screenshots.
Happy? :-p
#21
in reply to:
↑ 19
@
15 years ago
Replying to caesarsgrunt:
Replying to scribu:
A little more details wouldn't hurt: browser, os, which admin page(s) etc.
Browser: Firefox 3.5, Safari 4.0, Chrome 4.0...
OS: Doesn't make any difference, but if you're interested it's OS X 10.6.2.
Which pages: See the original ticket and screenshots.
Happy? :-p
Yes, thanks. :P
#23
in reply to:
↑ 17
@
15 years ago
Replying to caesarsgrunt:
Still getting square buttons in 2.9.1.
Yes, I've cleared my cache.
Perhaps you're caught in the same "strange" behaviour that brought this bug. Quitting FF and deleting the cache manually would fix that.
Dashboard (all round)