Opened 18 months ago
Closed 18 months ago
#19410 closed defect (bug) (fixed)
"New" in + New icon on Admin Bar is black on focus
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.3 |
| Component: | Toolbar | Version: | 3.3 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: |
Description
Related to: #19404
The text is black and hidden on :focus.
Attachments (2)
Change History (10)
- Keywords reporter-feedback added
- Milestone changed from Awaiting Review to 3.3
linuxologos — 18 months ago
comment:2
linuxologos — 18 months 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.
koopersmith — 18 months ago
comment:3
follow-up:
↓ 6
koopersmith — 18 months 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.
- Keywords commit added; needs-testing removed
I can confirm the bug. Not completely read up on the selectors, but and 19410.2.diff looks good.
comment:6
in reply to:
↑ 3
linuxologos — 18 months 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?