"Search Rows..." improperly colors red when you search for and find a column

Issue #390 resolved
Robert Leach created an issue

USE CASE: WHAT DO YOU WANT TO DO?

Search for a column without searching for a row first and get results that do not indicate a non-match (i.e. white search boxes)

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

  1. Open small_133x133.txt
  2. Type command-f (focus will change to 'search rows...')
  3. Type a tab (focus will change to 'search cols...')
  4. Type "test"
  5. Tap the down arrow key twice & hit return

CURRENT BEHAVIOR

  1. The row search box (containing the text "Search Rows...") turns red.
  2. The first 4 characters of "1,3-beta-glucan synthase complex|GO|081105" (including the quotes) in the column search box are selected/highlighted.
  3. The correct single entire column for that label is highlighted (along with the correct complement of labels).

EXPECTED BEHAVIOR

  1. The row search box remains white
  2. The entire "1,3-beta-glucan synthase complex|GO|081105" string is selected/highlighted (including the quotes)
  3. Same - no change in behavior of the matrix/labels

DEVELOPERS ONLY SECTION

SUGGESTED CHANGE (Pseudocode optional)

  1. Don't color the row search box red when the default text ("Search Rows...") is selected.
  2. The partial selection appears to occur only when selecting a drop-down item by the return key. If you click it, the entire string gets selected. So this selection behavior needs to be corrected where the return key is captured.

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

HeaderFinderBox

LEVEL OF EFFORT - developers only

minor

COMMENTS

Comments (4)

  1. Robert Leach reporter

    This resolves issue #390. The companion box's text value is now tested for a manual entry by comparing its value with the default text for that specific box's version of the default text.

    This was accomplished by removing the static qualifier from the variable declaration.

    → <<cset 3bfbcb23710d>>

  2. Robert Leach reporter

    This resolves issue #390. The companion box's text value is now tested for a manual entry by comparing its value with the default text for that specific box's version of the default text.

    This was accomplished by removing the static qualifier from the variable declaration.

    → <<cset 3bfbcb23710d>>

  3. Log in to comment