Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#44136 closed defect (bug) (invalid)

Privacy Pointer covers the thing it's pointing at

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.9.6
Component: Administration Keywords:
Focuses: ui, administration Cc:

Description

After upgrading to 4.9.6, a "Personal Data and Privacy" pointer appears to draw attention to the "Tools" menu, and the new submenu items.

Unfortunately, the pointer covers the Tools submenu that it's pointing at, which makes this pointer feel incomplete, unintentional, or broken.

Screenshot imminent.

Attachments (3)

Screen Shot 2018-05-17 at 4.20.50 PM.png (141.3 KB) - added by johnjamesjacoby 5 years ago.
Screen Shot 2018-05-17 at 4.46.53 PM.png (183.1 KB) - added by johnjamesjacoby 5 years ago.
Pixels
Screen Shot 2018-05-17 at 5.04.48 PM.png (114.6 KB) - added by johnjamesjacoby 5 years ago.
Quick mock-up with minimal improvements

Download all attachments as: .zip

Change History (7)

#1 @johnjamesjacoby
5 years ago

In my opinion, this is not a good use of the pointer UI element for the following reasons:

  • As noted above, z-index clashes make this an unreliable way to greet users after an update
  • The pointer protrudes, but the admin-menu is concave - together, it's an odd look
  • The overlapping arrows are not pixel-perfect - the pointer is about 2px too high
  • The pointer has 3 headings and only 2 paragraphs
  • The Settings > Privacy breadcrumb gets cut-off due to the limited pointer width - confusing at a glance, and because pointers really are supposed to be glanceable UI elements, I think it's failed here

@johnjamesjacoby
5 years ago

Quick mock-up with minimal improvements

#2 @johnjamesjacoby
5 years ago

My mock-up does a few things:

  • Adjusts the pointer arrow so the pointer can be more top-oriented
  • Adjusts the position of the pointer to be closer to the admin menu, to avoid the double arrow problem
  • Remove and reduce text to the minimum that it takes to clearly convey what's new

This ticket was mentioned in Slack in #core-privacy by desrosj. View the logs.


4 years ago

#4 @desrosj
4 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Thanks for all the information and effort here, @johnjamesjacoby. #45999 is aiming to remove the pointer in 5.2, making these changes unnecessary.

Note: See TracTickets for help on using tickets.