[wp-trac] [WordPress Trac] #47491: Avoid double colouring in privacy request rows w/ notices
WordPress Trac
noreply at wordpress.org
Fri Jul 19 23:37:27 UTC 2019
#47491: Avoid double colouring in privacy request rows w/ notices
-----------------------------------+--------------------------------
Reporter: garrett-eclipse | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: needs-design-feedback | Focuses: ui, accessibility
-----------------------------------+--------------------------------
Changes (by garrett-eclipse):
* keywords: needs-design => needs-design-feedback
Comment:
Replying to [comment:2 afercia]:
> Discussed during today's accessibility bug scrub and agreed this seems a
mix of accessibility, usability, and UI consistency issues.
Thanks @afercia I appreciate you and the rest of the Design team
discussing that today.
To answer some of the questions raised;
> - Why the notice is displayed within the table row in the first place?
Seems that’s unprecedented in WordPress.
Yes I believe this is unprecedented and was placed there to tie the notice
to the specific row that the ajax erase action was done on. Looking at the
process I almost feel a reload would make sense as currently after doing
the erase the button persists until a reload when it becomes 'Remove
request'. And moving the notice to it's normal position seems appropriate
but would want to have the Requester email/username added to it for
reference.
> - That shade of green and that shade of blue are too similar.
> - Blue is for 'information' it's saying 'something happened'. Which
isn't great.
> - Why are two actions happening simultaneously? Maybe that's the issue?
> - Color shouldn't be the only signifier of state...
> - There is a lot of unexpected here:
> - Double status colors
> - Embedded messages
Probably best to indicate why the row is blue here and get thoughts on
that. The row only become blue when a user confirms their request as a way
to indicate to the admin this row requires an action (The erasure). The
row already has the 'Confirmed' status indicated on it and the 'Confirmed'
list filter has the count on it. I'm not against just dropping the blue as
the admin will see the status as well as the Next Steps erasure button and
they received an email that the requester confirmed so they should be
aware.
With that in mind going to move this back to design review with the
suggested actions to be discussed being;
1. Removing the blue on confirmed request rows.
2. Moving the notice to it's normal position and adding the requestor
username/email to it. So it would be 'No personal data was found for
john at doe.com'
*I didn't bother mocking up as it would just use the standard notice
design and position.
Thoughts?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/47491#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list