Opened 11 months ago
Last modified 10 months ago
#49603 reopened feature request
Privacy and Ergonomics
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-patch needs-testing has-screenshots |
Focuses: | ui, accessibility | Cc: |
Description
Export Personal Data admin page and Erase Personal Data are very very similar which can lead to some manipulation errors for short-sighted and disabled people.
The following proposition aims to fix this issue
Attachments (10)
Change History (21)
#1
@
11 months ago
Am I correct in saying that the only change proposed is to allow the user to change the background colour?
#2
follow-up:
↓ 3
@
11 months ago
- Resolution set to invalid
- Status changed from new to closed
The privacy tools make use of the same stylesheet as the rest of /wp-admin/
This suggestion would be more appropriate for the Accessibility team.
(That way Plugins / Users screens, etc. could also have different background colours.)
#3
in reply to:
↑ 2
@
11 months ago
Replying to carike:
The privacy tools make use of the same stylesheet as the rest of /wp-admin/
Yep, alternate background color(s) should be implemented "globally" for wp-admin.
This suggestion would be more appropriate for the Accessibility team.
Don't think Accessibility (and the accessibility team) has anything to do with having alternate colors, alternate background colors, alternate border colors, etc. etc. They are implemented as "admin color themes" on the user profile screen and generally fall under "UI/design".
#4
@
11 months ago
Thanks @azaozz :)
I only suggested Accessibility because the reason given for the proposed change was to increase contrast / help users distinguish between pages, specifically to help people who are near-sighted.
UI/Design should be best though. :sendscoffee:
#5
@
11 months ago
- Component changed from General to Administration
- Focuses ui added
It looks like the background color is proposed here as a way to highlight the destructive nature of the Erase Personal Data screen and avoid an accidental erasure.
This might be an idea worth considering, but it would need to be done consistently across the admin, see #36882.
Tangentially related: #11304, #27370, #28235, #37016, #39712.
#6
@
10 months ago
@SergeyBiryukov
It looks like the background color is proposed here as a way to highlight the destructive nature of the Erase Personal Data screen
yes this is exactly the point.
So, you believe "trash" admin pages should also be optionally differentiated ?
That is a good point. 100% with you on this
#7
@
10 months ago
- Focuses accessibility added
- Keywords has-patch needs-testing needs-privacy-review has-screenshots added
- Resolution invalid deleted
- Status changed from closed to reopened
done !
would be nice !
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
10 months ago
#10
@
10 months ago
The accessibility team reviewed this today, and opened a related ticket - #49714 - to address this issue on a broader scale.
The color-only nature of this patch is of concern to us, although it's not directly an accessibility violation, given that the text on the page does indicate the destructive nature of the page - however, using a different color background selectively in a few contexts could be confusing, and could introduce unexpected contrast issues. We'd like to explore alternate solutions.
#11
@
10 months ago
I think only specific elements here should be highlighted as destructive, not the whole screen:
- "Erase Personal Data" link in the Next Steps column.
- "Force Erase Personal Data" row action.
Marking them with a red color would be consistent with other UI elements in the admin, e.g. various "Delete", "Spam", or "Trash" links. Adding a confirmation modal if it doesn't exist already would also be valuable.
Export Personal Data