![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 15 years ago
Closed 15 years ago
#19410 closed defect (bug) (fixed)
"New" in + New icon on Admin Bar is black on focus
| Reported by: | linuxologos | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.3 |
| Component: | Toolbar | Version: | 3.3 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
Description
Related to: #19404
The text is black and hidden on :focus.
Attachments (2)
Change History (10)
#2
@
15 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
@
15 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
@
15 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
@
15 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?