reviewboard: remove hover/reveal effects when viewing diffs (
bug 1245234) r?davidwalsh
The controls for expanding diff context are hidden with a slow javascript
animation to display. Remove this pain point.
Function is commented out instead of removed to minimise merge conflicts.
MozReview-Commit-ID: DuLw0U4PiQH
--- a/reviewboard/reviewboard/static/rb/js/views/diffFragmentQueueView.js
+++ b/reviewboard/reviewboard/static/rb/js/views/diffFragmentQueueView.js
@@ -310,22 +310,24 @@ RB.DiffFragmentQueueView = Backbone.View
this._tryHideControlsDelayed($expanded);
},
/*
* When a comment container loads its contents for the first time, hide its
* controls and set up the mouseenter and mouseleave events.
*/
_onFirstLoad: function(id) {
+ /* MozReview: always show controls
var $container = this.$('#' + this.options.containerPrefix + '_' + id);
this._hideControls($container, true);
$container.hover(_.partial(this._tryShowControlsDelayed, $container),
_.partial(this._tryHideControlsDelayed, $container));
+ */
},
/*
* Attempt to hide the controls in the given container after a delay.
*/
_tryShowControlsDelayed: function($container) {
_.delay(_.bind(function() {
if ($container.is(':hover')) {