• 
      

    Remove hoverIntent dependancy

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

    Information

    Review Board
    release-2.5.x

    Reviewers

    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:
    Completed
    Change Summary:
    Pushed to release-2.5.x (9aa2199)