Opened 8 weeks ago
Last modified 4 weeks ago
#62866 new enhancement
Adjust background alternate row on post and page tables
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 6.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | needs-design-feedback has-patch |
Focuses: | ui, accessibility, css | Cc: |
Description (last modified by )
Now that the lighter background has been added, the background on hover for the post and pages table could be adjusted also. Whilst the lightest color grey is #f0f0f0 in Storybook, dataViews have the table hover at #f8f8f8. I would recommend we use that and attach that as option.
What this does is reduce number of grays which are getting out of control a bit again.
Attachments (2)
Change History (18)
This ticket was mentioned in PR #8194 on WordPress/wordpress-develop by @karmatosed.
8 weeks ago
#1
- Keywords has-patch added; needs-patch removed
#3
@
8 weeks ago
I am uploading the existing stripe alt to show how small the change is but how beneficial it is in reducing yet more differences.
@audrasjb commented on PR #8194:
8 weeks ago
#4
#5
@
8 weeks ago
What are the current and new color contrast ratios between the background color and the different text colors? Since the text color did not change.
#6
@
7 weeks ago
@poena it's better than before since the odd rows background color is a bit lighter than before: now we have 6.89:1 for normal text and 4.86:1 for blue links.
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
6 weeks ago
@karmatosed commented on PR #8194:
6 weeks ago
#9
@audrasjb I like idea of that, let me see about working on those.
This ticket was mentioned in Slack in #accessibility by audrasjb. View the logs.
5 weeks ago
#11
follow-up:
↓ 13
@
5 weeks ago
I think we need to work on adapting the core color palette (https://make.wordpress.org/core/2021/02/23/standardization-of-wp-admin-colors-in-wordpress-5-7/) so that we shift to the new palette consistently.
E.g., if dataviews are using #f8f8f8
, the closest in core colors is #f6f7f7
, and we should make those agree.
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
4 weeks ago
#13
in reply to:
↑ 11
@
4 weeks ago
Totally agree, we ought to have a systematic approach with the least amount effort in order to update in a clean way, having color #f6f7f7 still works as the contrast ratio still reaches the AA rank (not the AAA one but I think that's ok).
Replying to joedolson:
I think we need to work on adapting the core color palette (https://make.wordpress.org/core/2021/02/23/standardization-of-wp-admin-colors-in-wordpress-5-7/) so that we shift to the new palette consistently.
E.g., if dataviews are using
#f8f8f8
, the closest in core colors is#f6f7f7
, and we should make those agree.
This changes the background color to be consistent with the one used on hover in dataViews.
What this does is reduce number of grays which are getting out of control a bit again.
Trac ticket: https://core.trac.wordpress.org/ticket/62866.
---
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request.
See GitHub Pull Requests for Code Review in the Core Handbook for more details.