- marked as trivial
Simplify the double-click MouseListener code for zooming
USE CASE: WHAT DO YOU WANT TO DO?
Make the code more easily understandable.
STEPS TO REPRODUCE AN ISSUE (OR TRIGGER A NEW FEATURE)
Read the MouseListener code for handling clicks in the IMV.
CURRENT BEHAVIOR
Edge cases were applied to the code after the basics were laid out and without optimal integration into existing logic.
EXPECTED BEHAVIOR
The logic should be more holistic - check fewer variables in each if conditional and possibly even create methods to handle them.
DEVELOPERS ONLY SECTION
SUGGESTED CHANGE (Pseudocode optional)
FILES AFFECTED (where the changes will be implemented) - developers only
Currently, it's InteractiveMatrixView.java, but that might change after pull request 14.
LEVEL OF EFFORT - developers only
medium
COMMENTS
Check out the timer used for the thumbs in the color bar to detect double-clicks (colorchoosercontroller)
Comments (3)
-
reporter -
reporter - changed component to Coding Convention Adherence
-
reporter - removed milestone
Removing milestone: Coding Convention Adherence (automated comment)
- Log in to comment