WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#13136 closed enhancement (fixed)

Menus without JS: remove "move" cursor

Reported by: xibe Owned by: filosofo
Milestone: 3.0 Priority: normal
Severity: minor Version: 3.0
Component: Menus Keywords: has-patch
Focuses: Cc:

Description

When disabling JavaScript, the latest menu does display arrows to move an item up/down a specific menu - neat!
But the whole <dt> tag still features the cursor:move CSS rule, which doesn't make sense since you can't move the item any more.
In order to remain coherent, I suggest this CSS line be removed when JS is deactivated.

Attachments (1)

13136.1.patch (572 bytes) - added by koopersmith 5 years ago.

Download all attachments as: .zip

Change History (4)

@koopersmith5 years ago

comment:1 @koopersmith5 years ago

This patch fixes the issue and also compensates for the missing whitespace between menu items in the no-js version (due to the lack of droppable divs).

comment:2 @koopersmith5 years ago

  • Keywords has-patch added

comment:3 @ryan5 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [14251]) Remove move cursor when JS disabled. Props koopersmith. fixes #13136

Note: See TracTickets for help on using tickets.