Exception having to do with headerFinderBox when data dimensions are different

Issue #282 closed
Robert Leach created an issue

USE CASE: WHAT DO YOU WANT TO DO?

Select a different header in the label dialog

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

  1. Open John Matese's 11.cdt file in the test data
  2. Select View->Labels
  3. Click UID in the rows pane

CURRENT BEHAVIOR

Exception is thrown about headers being out of bounds and the UID label type is not highlighted. Further attempots to open the label settings results in no label setting window appearing.

EXPECTED BEHAVIOR

A new row label type is selected and everything works.

DEVELOPERS ONLY SECTION SUGGESTED CHANGE (Pseudocode optional)

Quick-fix for alpha03: The "other" dimension's selection is only used to determine whether the search results are visible, so the headers aren't actually necessary. Eliminate them and rely on the functions provided by the otherSelection object.

Long-term-fix: See issue #252.

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

HeaderFinderBox DendroView

LEVEL OF EFFORT - developers only

minor

COMMENTS

This issue was split off from issue #252

Comments (5)

  1. Robert Leach reporter

    I split this issue off from issue #252. This specific portion of that issue is now fixed. I will submit a commit request. The effort was minor, so feel free to forgo a document review meeting and accept/reject the changes as-is @TreeView3Dev. I'll include a diff.

  2. Log in to comment