Fix the DiffCommitIndexView to no longer use hoverIntent
Review Request #7652 — Created Sept. 20, 2015 and submitted
The hoverIntent jQuery plugin was removed as a dependency on the
release-2.5.x
branch and that change was eventually merged into the
dvcs
branch. However, theDiffCommitIndexView
utilized this plugin.
The view has been reworked to no longer depend on it and a bit of
cleanup has occurred.
Verified that the summaries expanded in the diff viewer.
Description | From | Last Updated |
---|---|---|
Can we keep the -s and +s lined up with the other parts of the equation instead of the =? |
david |
- Description:
-
~ The hoverIntent jQuery plugin was removed as a dependency on the
release-2.5.x
branch and that change was eventually merged into thedvcs
branch. However, theDiffCommitIndexView
utilized this plugin. The view has been reworked to no longer depend on it and a bit of cleanup has occurred.~ The hoverIntent jQuery plugin was removed as a dependency on the
+ release-2.5.x
branch and that change was eventually merged into the+ dvcs
branch. However, theDiffCommitIndexView
utilized this plugin.+ The view has been reworked to no longer depend on it and a bit of + cleanup has occurred.
-
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/diffviewer/views/diffCommitIndexView.js reviewboard/static/rb/js/diffviewer/models/diffCommitModel.js Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/diffviewer/views/diffCommitIndexView.js reviewboard/static/rb/js/diffviewer/models/diffCommitModel.js