Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#33290 closed defect (bug) (fixed)

Nav Menus quick search spinner position

Reported by: afercia's profile afercia Owned by: afercia's profile afercia
Milestone: 4.4 Priority: normal
Severity: normal Version: 4.2
Component: Menus Keywords: has-patch has-screenshots
Focuses: ui Cc:

Description

Some (old) operating systems have scrollbars that take space and don't use "overlay scrollbars". When searching items in the nav menus screen, XP, Vista and Windows 7 users will see a misplaced spinner because scrollbars take space. See screenshot below, from left to right: latest Chrome, Firefox and IE 8

https://cldup.com/XeMKEMeeAT.png

When the spinner disappears, it will still affect layout and that big empty space below the input field will still be there.

I guess this doesn't happen on Windows 8, modern Linux distros, Macs, iOS, Android, etc. since they all use "overlay scrollbars".

Attachments (1)

33290.patch (358 bytes) - added by afercia 9 years ago.

Download all attachments as: .zip

Change History (8)

@afercia
9 years ago

#1 @afercia
9 years ago

  • Keywords has-patch added

After the patch:

https://cldup.com/UpSmAyra7E.png

#2 @afercia
9 years ago

  • Milestone changed from Awaiting Review to 4.4
  • Owner set to afercia
  • Status changed from new to assigned

#3 @afercia
9 years ago

To reproduce (Windows XP, Vista, 7):

  • do a first search and be sure to get enough results to make the scrollbar appear
  • notice the empty space below the search field
  • do a second search
  • notice the spinner displayed under the search field

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


9 years ago

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


9 years ago

#6 @afercia
9 years ago

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

In 34065:

Nav Menus: fix the quick search spinner position on Operating Systems that show scrollbars.

Fixes #33290.

#7 @ryan
9 years ago

  • Keywords has-screenshots added
Note: See TracTickets for help on using tickets.