#33808 closed defect (bug) (fixed)
Use the circular focus style in more places
Reported by: | helen | Owned by: | helen |
---|---|---|---|
Milestone: | 4.4 | Priority: | low |
Severity: | normal | Version: | 4.3 |
Component: | General | Keywords: | needs-patch |
Focuses: | ui, accessibility, administration | Cc: |
Attachments (1)
Change History (14)
@
9 years ago
Adds circular focus style to arrows on dashboard widgets, post edit metaboxes, nav-menus, and widgets.
#3
follow-up:
↓ 4
@
9 years ago
- Keywords has-patch added; needs-patch removed
Although this patch adds the circular focus style in all the places I could find, the css looks like a mess with stuff all over the place. Maybe we need a common pattern and shared css for these kind of expandable boxes.
#4
in reply to:
↑ 3
@
9 years ago
Replying to grvrulz:
Thanks for the patch, will take a look.
the css looks like a mess with stuff all over the place. Maybe we need a common pattern and shared css for these kind of expandable boxes.
Believe me, that's known. https://make.wordpress.org/core/2014/12/19/core-css-roadmap/
#5
@
9 years ago
@grvrulz Patch looks good around the admin, though the focus style should be applied to the same element as the icon instead of making buttons/links smaller than they were. Seems like the customizer needs work too, which I've already started to dig into so I'll finish that up as well.
#8
follow-up:
↓ 9
@
9 years ago
@helen Thanks for linking to the roadmap, it's an interesting read.
Also, Re: -webkit
, I haven't delved into this deeply, but don't we have an autoprefixer in the build script?
#9
in reply to:
↑ 8
@
9 years ago
Replying to grvrulz:
Also, Re:
-webkit
, I haven't delved into this deeply, but don't we have an autoprefixer in the build script?
It's not in build, we run it precommit and I neglected to do so this time.
#11
@
9 years ago
- Keywords needs-patch added; has-patch removed
Leaving this open because the text-indent: -1px
thing doesn't work in RTL. Not sure if that's something that RTLCSS really should handle, since people use it for hiding text and such, so probably just need to add some specific rules ourselves.
Can you point out a specific spot where this is used or the class used to call it?