Zoom selection button activation delayed/lagged

[Robert Leach]

USE CASE: WHAT DO YOU WANT TO DO?

Make a selection and see that the zoom selection button becomes active.

STEPS TO REPRODUCE AN ISSUE (OR TRIGGER A NEW FEATURE)

1. Open John Matese's 11.cdt file
2. Type command-f to give the row search box focus
3. Click on a tile on the matrix to select it
4. Hover over the zoom selection button
5. Deselect the data by right-clicking anywhere on the data
6. Hover over the zoom selection button

CURRENT BEHAVIOR

The zoom selection button status is not updated immediately upon selection change.

EXPECTED BEHAVIOR

When you make a selection, the zoom selection button should be enabled immediately upon mouseReleased. When you deselect (by right clicking or otherwise), the zoom selection button should be disabled immediately upon mouseReleased.

DEVELOPERS ONLY SECTION

SUGGESTED CHANGE (Pseudocode optional)

none

FILES AFFECTED (where the changes will be implemented) - developers only

unknown

LEVEL OF EFFORT - developers only

minor

COMMENTS

This might be specific to the data file, since it seems to work with other data files.

[Robert Leach]

See attached screen recording of the behavior in action.

[Robert Leach]

I cannot reproduce this behavior on my iMac. I believe I first encountered it on my laptop. I should retry it on the laptop, but for now, I'm chalking this up as a fluke and putting this issue on hold.

[Robert Leach]

I was able to reproduce this issue again off the master branch on my iMac. It starts happening only after you have performed a search. Here's another screen recording:

https://bitbucket.org/TreeView3Dev/treeview3/downloads/selec_button_lag.mov

[Robert Leach]

Updated to add steps to the procedure to reproduce the error.

[Robert Leach]

OK, it's only when a search box has focus. You do not have to perform a search to make it happen.

[Robert Leach]

Removed steps from the procedure to reproduce the behavior (a search box only needs focus - no search required).

[Robert Leach]

Starting work.

[Robert Leach]

Fixed in branch issue415...

[Robert Leach]

Resolves issue #415. The zoom selection button now activates immediately upon selection change even if focus is on a search box.

Had to move the call to setAdaptiveButtonStatus to the top of the method for requesting focus on the zoom selection button because it was returning immediately if focus is on a search box. As I recall, that return was added so that a user can make multiple searches without having to tab back to the search box each time.

→ <<cset 0cf4076fb4e8>>

[Robert Leach]

Resolves issue #415. The zoom selection button now activates immediately upon selection change even if focus is on a search box.

Had to move the call to setAdaptiveButtonStatus to the top of the method for requesting focus on the zoom selection button because it was returning immediately if focus is on a search box. As I recall, that return was added so that a user can make multiple searches without having to tab back to the search box each time.

→ <<cset 0cf4076fb4e8>>

[Robert Leach]

Removing milestone: Selecting Data (automated comment)