Remove hoverIntent dependancy

Review Request #7550 — Created July 28, 2015 and submitted

brennie
Review Board
release-2.5.x
reviewboard

The RB.DiffFragmentQueueView has been refactored to no longer depend
on the hoverIntent jQuery plugin. This plugin adds mousemove event
listeners to the page. In lieu of this behaviour, the
RB.DiffFragmentQueueView now just checks if the given jQuery object
has the :hover selector.

The view has also been refactored to not use its own mousemove
listeners and to debounce the resize event.

The move to checking selectors has reduced the code complexity and
made it easier to follow.

Verified the controls still worked as expected.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/static/lib/js/jquery.hoverIntent.js
        reviewboard/static/rb/js/views/diffFragmentQueueView.js
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/static/lib/js/jquery.hoverIntent.js
        reviewboard/static/rb/js/views/diffFragmentQueueView.js
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
brennie
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.5.x (9aa2199)
Loading...