Page MenuHomePhabricator

Popups appear too quickly (resulting in a lot of accidental hovers)
Closed, ResolvedPublic

Description

Popups appear too quickly (resulting in a lot of accidental hovers), for me at least. Maybe I'm just weird.

Looking at the code, the delay is 150ms (or 50ms + download time) for opening the popup and 100ms for closing it. The original Navigation Popups use 500ms for both.


Version: master
Severity: normal

Details

Reference
bz64234

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:08 AM
bzimport added a project: Page-Previews.
bzimport set Reference to bz64234.

(In reply to Bartosz comment #0)

The original Navigation Popups use 500ms for both.

If I remember correctly, it also lets the user change this number.

Maybe I'm just weird.

Maybe that. Or, we all are. We are logging the time duration of the Hovercard being open. I have requested some analytics [1] on it to be able to fine tune the delay and reduce accidental hovers.

[1] https://trello.com/c/Sbd8Q180/99-events-for-hovercards-beta-feature

(In reply to Prateek Saxena from comment #1)

(In reply to Bartosz comment #0)

The original Navigation Popups use 500ms for both.

If I remember correctly, it also lets the user change this number.

The configuration is explained here https://en.wikipedia.org/wiki/Wikipedia:NAVPOP#Options with popupDelay being the top item. (I'm not sure where exactly the code is)
It might be good to reuse that variable name, so if an editor customizes one, it works for both.

(In reply to Quiddity from comment #2)

It might be good to reuse that variable name, so if an editor customizes
one, it works for both.

Should we use it override the current setting wherever available. I know that is what you are saying but I just want to confirm once before implementing.

(In reply to Prateek Saxena from comment #3)

(In reply to Quiddity from comment #2)

It might be good to reuse that variable name, so if an editor customizes
one, it works for both.

Should we use it override the current setting wherever available. I know
that is what you are saying but I just want to confirm once before
implementing.

If I understand correctly, then yes. For example: I have this code in my [[User:Quiddity/vector.js]] -

// [[User:Lupin/popups.js]]
popupDelay=0.3;
popupHideDelay=0.3;

(because I wanted it to be ever-so-slightly faster than the default of 0.5)

and other people use a wide range of numbers - as I wrote months ago, on the archived talkpage:
If we search Enwiki for the parameter name,[1] we see ~740 users have tweaked their settings, with a wide-range of figures, from 0 to 10 seconds. (It might be good to analyze those customizations for patterns?)
[1] https://en.wikipedia.org/w/index.php?title=Special:Search&limit=100&offset=0&redirs=1&ns2=1&search=popupDelay

Change 137897 had a related patch set uploaded by Prtksxna:
renderer: Use NavigationPopups settings where available

https://gerrit.wikimedia.org/r/137897

(In reply to Quiddity from comment #4)

and other people use a wide range of numbers - as I wrote months ago, on the
archived talkpage:
If we search Enwiki for the parameter name,[1] we see ~740 users have
tweaked their settings, with a wide-range of figures, from 0 to 10 seconds.
(It might be good to analyze those customizations for patterns?)
[1] https://en.wikipedia.org/w/index.php?title=Special:Search&limit=100&offset=0&redirs=1&ns2=1&search=popupDelay

This is rather interesting. I looked at the results and it seems like most of the customizations are raising popupDelay or lowering popupHideDelay.

Therefore I propose values of, respectively, 500ms and 300ms for Hovercards, especially since according to bug 66301 I am not the only person to think the current ones are too quick.

  • Bug 66301 has been marked as a duplicate of this bug. ***

Change 140917 had a related patch set uploaded by Bartosz Dziewoński:
Raise popup show/hide delay to 500ms/300ms, respectively

https://gerrit.wikimedia.org/r/140917

Change 140917 merged by Prtksxna:
Raise popup show/hide delay to 500ms/300ms, respectively

https://gerrit.wikimedia.org/r/140917