Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#19575 closed defect (bug) (fixed)

Never show two flyouts at once

Reported by: nacin's profile nacin Owned by: ryan's profile ryan
Milestone: 3.3.1 Priority: normal
Severity: trivial Version: 3.3
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

There is an intentional (and slight) delay until a flyout gets hidden. But the moment another one is opened, the delay should be terminated and the flyout should immediately be closed. Two flyouts should never show at the same time.

This bug has probably contributed to the occasionally voiced perception that the dashboard is a bit slower in 3.3.

Attachments (2)

capture.png (6.5 KB) - added by johnbillion 12 years ago.
19575.patch (393 bytes) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (17)

#1 @knutsp
12 years ago

  • Cc knut@… added

I'm not able to reproduce this bug. I move from one main menu item to another very quickly (moving mouse pointer vertically) and are not able to see two flyouts at once. Or may be for 1/100 of a second, wich could be a browser issue to me. Hardly a contribution to a perception of a slow dashboard. Suggest "worksforme".

#2 @ocean90
12 years ago

knutsp, just hover fast over media and links again and again, then you can see it.

#3 @knutsp
12 years ago

  • Severity changed from normal to trivial

After extensive testing I am able to move from Links to Media and back, without the Links flyout closing and the Links flyout being present above it for a split second or less.

I can hardly see this as a bug, even a very minor one. If I am wrong, severity less than trivial.

#4 @knutsp
12 years ago

  • Version set to 3.3

@johnbillion
12 years ago

#5 @johnbillion
12 years ago

+1. It's only a slight delay but it gives the impression of lag.

#6 @Utkarsh
12 years ago

  • Cc utkarshkukreti@… added

#7 @ctsttom
12 years ago

  • Cc me@… added

I have tested this on my setup and was able to reproduce, here is a video:
http://youtu.be/2wnP-is5OqQ

Running Chrome 16.0.912.63 on Mac OS X Lion 10.7.2 (on a high end iMac).

#8 @SergeyBiryukov
12 years ago

  • Keywords has-patch added

#9 @nacin
12 years ago

Patch looks good, but what do I know. koopersmith?

#10 @azaozz
12 years ago

19575.patch looks good here too, makes the flyouts a bit snappier.

#11 @ryan
12 years ago

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

In [19647]:

Immediately close a flyout when opening another one. Prevents brief occurrence of two flyouts at once. Props SergeyBiryukov. fixes #19575 for trunk

#12 @ryan
12 years ago

In [19648]:

Immediately close a flyout when opening another one. Prevents brief occurrence of two flyouts at once. Props SergeyBiryukov. fixes #19575 for 3.3

#13 follow-up: @unknowndomain
11 years ago

Can someone test this again, because it looks like it is already fixed from my testing.

#14 in reply to: ↑ 13 ; follow-up: @helen
11 years ago

Replying to unknowndomain:

Can someone test this again, because it looks like it is already fixed from my testing.

Ticket is marked fixed. :)

#15 in reply to: ↑ 14 @unknowndomain
11 years ago

Replying to helen:

Replying to unknowndomain:

Can someone test this again, because it looks like it is already fixed from my testing.

Ticket is marked fixed. :)

Such an idiot!

Note: See TracTickets for help on using tickets.