Diff expansion controls now have a delay before appearing.
Review Request #6583 — Created Nov. 13, 2014 and submitted
Diff fragment expansion controls will only expand if the mouse hovers
over the diff fragment for a half of a second. This unifies behaviour
in Firefox in Chrome so that when you scroll down a review request,
the diff fragment controls won't expand as you scroll by.Scrolling to a diff fragment and stopping now works correctly in both
Firefox and Chrome. That is, the diff fragment will expand as long as
the mouse stays it for half of a second.The
diffFragmentQueueViewnow maintains the number of mouse watchers
so that multiple mouse move event handlers are not added and the event
handler is only removed once there are no active mouse watchers. This
behaviour is used in the case of multiple diff fragment expansion
events and/or scrolling over diff fragments occuring near the same
time.
Verified in Firefox, Chrome, and Safari.
| Description | From | Last Updated |
|---|---|---|
|
I know it was this way before, but can you change this to: this._mouse = { x: ..., y: ..., … |
 chipx86
|
|
|
Given the number of arguments to these functions, can you format this more like: _.delay( _.bind(function() { ... }, this), … |
chipx86
|
|
|
There's an extra space before "mouse". |
chipx86
|
|
|
These should be lowercase. |
chipx86
|
-
-
reviewboard/static/rb/js/views/diffFragmentQueueView.js (Diff revision 1) I know it was this way before, but can you change this to:
this._mouse = { x: ..., y: ..., ... }; -
reviewboard/static/rb/js/views/diffFragmentQueueView.js (Diff revision 1) Given the number of arguments to these functions, can you format this more like:
_.delay( _.bind(function() { ... }, this), this._timeout); -
reviewboard/static/rb/js/views/diffFragmentQueueView.js (Diff revision 1) There's an extra space before "mouse".
-
reviewboard/static/rb/js/views/diffFragmentQueueView.js (Diff revision 1) These should be lowercase.
Change Summary:
Address issues in Christian's review.
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+79 -27) |
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/diffFragmentQueueView.js Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/diffFragmentQueueView.js