WordPress.org

Make WordPress Core

Opened 10 months ago

Last modified 6 weeks ago

#51551 new defect (bug)

Unclickable area within admin menu

Reported by: johnbillion Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Administration Keywords: good-first-bug has-screenshots has-patch reporter-feedback
Focuses: ui, css Cc:

Description

When a top level admin menu item is expanded, for example when viewing the Posts screen, there is a dead area immediately below the top level menu item and its first child menu item which is unclickable but still shows the pointer cursor.

I occasionally click this area and wonder why my click didn't register.

Steps to reproduce:

  1. Click the Posts menu
  2. Hover your mouse cursor immediately below the top level Posts item but above the All Posts item
  3. Observe that your cursor shows a pointer but this area is not clickable

See attached screenshots for more info.

Attachments (9)

Screenshot 2020-10-16 at 11.10.57.png (33.9 KB) - added by johnbillion 10 months ago.
Screenshot 2020-10-16 at 11.11.26.png (25.3 KB) - added by johnbillion 10 months ago.
51551.patch (429 bytes) - added by opurockey 9 months ago.
Apply patch
51551.1.patch (429 bytes) - added by opurockey 9 months ago.
Apply patch
admin-menu.png (44.1 KB) - added by ryelle 2 months ago.
51551.2.diff (379 bytes) - added by kurudrive 6 weeks ago.
51551_1_ss.png (364.7 KB) - added by kurudrive 6 weeks ago.
51551.3.diff (362 bytes) - added by kurudrive 6 weeks ago.
admin-menu- 51551-3-diff.mov (802.4 KB) - added by kurudrive 6 weeks ago.

Download all attachments as: .zip

Change History (14)

#1 @johnbillion
10 months ago

  • Keywords has-screenshots added

This ticket was mentioned in Slack in #core by ankurchauhan126. View the logs.


9 months ago

This ticket was mentioned in PR #685 on WordPress/wordpress-develop by colinduwe.


9 months ago

  • Keywords has-patch added; needs-patch removed

Removed unclickable top padding of sub-menu <ul>. Added top padding to <a> of first sub-menu <li> so it is clickable.

Trac ticket: https://core.trac.wordpress.org/ticket/51551

@opurockey
9 months ago

Apply patch

@opurockey
9 months ago

Apply patch

@ryelle
2 months ago

#4 @ryelle
2 months ago

  • Keywords reporter-feedback added

Now that the submenu items have a hover indicator, do you still think the gap is an issue? There's more visual feedback that you're not hovering over the item yet.

@kurudrive
6 weeks ago

@kurudrive
6 weeks ago

#5 @kurudrive
6 weeks ago

Hi everyone.

At first, I checked @opurockey 's 51551.1.patch.
I can looks like some improvement but still remaining probrem.

https://core.trac.wordpress.org/raw-attachment/ticket/51551/51551_1_ss.png

So I created another patch file.

https://core.trac.wordpress.org/attachment/ticket/51551/51551.3.diff

@johnbillion @ryelle
Please check when you have time.

Last edited 6 weeks ago by kurudrive (previous) (diff)

@kurudrive
6 weeks ago

Note: See TracTickets for help on using tickets.