Remove hoverIntent dependancy
Review Request #7550 — Created July 28, 2015 and submitted
The
RB.DiffFragmentQueueViewhas been refactored to no longer depend
on the hoverIntent jQuery plugin. This plugin addsmousemoveevent
listeners to the page. In lieu of this behaviour, the
RB.DiffFragmentQueueViewnow just checks if the given jQuery object
has the:hoverselector.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.
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