Opened 13 years ago
Closed 13 years ago
#19410 closed defect (bug) (fixed)
"New" in + New icon on Admin Bar is black on focus
Reported by: | linuxologos | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.3 | Priority: | normal |
Severity: | normal | Version: | 3.3 |
Component: | Toolbar | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
Related to: #19404
The text is black and hidden on :focus.
Attachments (2)
Change History (10)
#2
@
13 years ago
- Keywords has-patch added; needs-patch reporter-feedback removed
I can see that with IE/Firefox/Chrome (latest versions) on Windows 7 with latest trunk. I can test on Linux too, if necessary.
19410.patch fixes the problem for me, but I'm not sure if it's good CSS.
#3
follow-up:
↓ 6
@
13 years ago
- Keywords needs-testing added
Confirmed — you have to click (and hold) the "New" item on the front-end to notice the bug. It's important that we only fix this for when jQuery is disabled, as the focus styles differ.
attachment:19410.2.diff addresses this issue and the simplest selector in the correct place.
#4
@
13 years ago
- Keywords commit added; needs-testing removed
I can confirm the bug. Not completely read up on the selectors, but 19410.2.diff looks good.
#6
in reply to:
↑ 3
@
13 years ago
Replying to koopersmith:
Confirmed — you have to click (and hold) the "New" item on the front-end to notice the bug. It's important that we only fix this for when jQuery is disabled, as the focus styles differ.
Yes, I was at the front-end with TwentyEleven, where jQuery is not enabled by default.
19410.2.diff works fine.
@linuxlogos: What browser/OS are you using?