#32294 closed defect (bug) (fixed)
Focus style: Switch View links
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.3 | Priority: | normal |
| Severity: | normal | Version: | 4.2 |
| Component: | General | Keywords: | has-patch |
| Focuses: | ui, accessibility | Cc: |
Description
Splitting this out from #29897. See screenshot before and after patch.
Attachments (2)
Change History (8)
#2
follow-up:
↓ 3
@
11 years ago
I think we should remove the 5px left margin from .view-switch > a + a to put both links a bit together again.
#3
in reply to:
↑ 2
@
11 years ago
Replying to ocean90:
I think we should remove the 5px left margin from
.view-switch > a + ato put both links a bit together again.
Yup, I'm also wondering if the child selector > is really needed in all the other rules, I don't see any reason for that. The previous patch missed the color change on focus, tried to simplify selectors and clean up a bit and all appears to work fine. Also, the clickable area should probably be a bit bigger on smaller screens, see screenshot below:
#4
@
11 years ago
In the refreshed patch:
- Fix the focus style for the Switch View links in the Posts listings and Media Library, remove margin between links, simplify CSS selectors and make the clickable area bigger on smaller screens.
Note: See
TracTickets for help on using
tickets.


In the proposed patch: