Lower the delay for expanding diff context on hover
Review Request #8474 — Created Oct. 18, 2016 and submitted — Latest diff uploaded
Previously, the code that handled expanding the diff context
accidentally waited twice the timeout due to two calls to_.delay
,
possibly from a refactoring that occurred to remove the $.hoverIntent
dependency. This has been removed and the delay has been lowered to
250ms (from 500ms) to make the UI feel much snappier. It used to take
over 1s to fully expand controls (500ms for each_.delay
call plus
the length of jQuery's "slow" animation). It now takes just over half
that to fully expand them.
Hovered over fragments and saw the delay was reduced.