Simplify the double-click MouseListener code for zooming

Issue #305 new
Robert Leach created an issue

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)

  1. Robert Leach reporter
    • removed milestone

    Removing milestone: Coding Convention Adherence (automated comment)

  2. Log in to comment