Long Press timeout is too long |
||||||||||||||||
Issue descriptionOn Chrome OS, the long press affordance is gone.
,
May 18 2016
I've seen it a few places now, ToT, and a fresh Chrome OS image 8339.0.0
,
May 18 2016
Dev has the issue.
,
May 18 2016
I can repro on 52.0.2733.0 on lulu jdufault@ I know you're swamped with issues for 54, but I've got no one else to look at this. Can you take a quick peek?
,
May 18 2016
,
May 18 2016
,
May 18 2016
See bug 158091. This was intentionally removed.
,
May 18 2016
The timeout seems very long without any visual indication that something is happening. Was there any discussion of shortening it?
,
May 19 2016
Owner to kuscher@ who requested/prioritized the removal (see linked bug 158091).
,
May 24 2016
Is this really a stable blocker for R51 at this point?
,
May 25 2016
Animation indeed removed intentionally. If we want to shorten timing, we should look at other platforms to see what the timeout there is Not a blocker
,
May 25 2016
From rough tests with a timer, it seems to be ~700ms on Android vs ~1400ms on Chrome OS. @jdufault, could we try out 700ms and see how that feels?
,
May 25 2016
I dug around the android source, it looks like 500ms[1] is the standard, but it can be overridden. 1: http://androidxref.com/6.0.1_r10/xref/frameworks/base/core/java/android/view/ViewConfiguration.java#64
,
Jun 1 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 15 2016
Issue 603642 has been merged into this issue.
,
Jun 15 2016
It would be really good to get this in for 53 because of Android apps. jdufault@ I know your plate is full. Are you going to have time to land this or should I try to find someone else?
,
Jun 15 2016
You can reassign to me if there is no one else and I'll prioritize this. The change should be very simple, just have to find what to change.
,
Jun 15 2016
warx@ can you take this one as a high priority?
,
Jun 15 2016
OK. I will take a look into it.
,
Jun 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ecd768b8eaa996dc371c67ffb9c3221808949712 commit ecd768b8eaa996dc371c67ffb9c3221808949712 Author: warx <warx@chromium.org> Date: Thu Jun 16 16:22:50 2016 Changing long press timeout 1000ms to 500ms The long press affordance is removed. Meanwhile, the long press time is still 1000ms which is too long. Change it to 500ms to align with standard. BUG= 612890 Review-Url: https://codereview.chromium.org/2064423003 Cr-Commit-Position: refs/heads/master@{#400167} [modify] https://crrev.com/ecd768b8eaa996dc371c67ffb9c3221808949712/ui/events/gesture_detection/gesture_configuration.cc
,
Jun 16 2016
changing long press time out config from 1000ms to 500ms.
,
Jul 18 2016
,
Apr 5 2017
,
Apr 5 2017
|
||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||
Comment 1 by tdander...@chromium.org
, May 18 2016Status: Assigned (was: Untriaged)