Opened 6 years ago
Closed 6 years ago
#2947 closed enhancement (done)
Configure Dialog for IndexedDataTableView
Reported by: | pfleck | Owned by: | pfleck |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.16 |
Component: | Analysis.Views | Version: | trunk |
Keywords: | Cc: |
Description
The IndexedDataTableView is missing some features that are implemented in the DatatTableView and in the ScatterPlotView.
- Configuring the Charts with a VisualPropertiesDialog
- Implementing IConfigureableView` for opening the dialog
- ContextMenuItem for opening the same dialog
- Hiding the Name-TextBox
- The corresponding properties of the DataTable- and ScatterPlotView should be named consisted (currently ShowChartOnly and ShowName).
Change History (15)
comment:1 Changed 6 years ago by pfleck
- Status changed from new to accepted
- Version set to branch
comment:2 Changed 6 years ago by pfleck
- Added interfaces for DataRow and DataTable (IDataTable is generic because the NamedItemCollection is invariant)
- Adapted DataTableVisualPropertiesDialog to use the new interfaces
- Added IConfigurableView to IndexedDataTableView
- Fixed VisualProperty.Title change in IndexedDataTableView
comment:3 Changed 6 years ago by pfleck
- Added context menu to open the VisualProperties dialog for the IndexedDataTableView.
- Changed the StatterPlotView.ShowName into ShowChartOnly to match the property names of the (Indexed)DataTableView.
comment:4 Changed 6 years ago by pfleck
r16156 fixed initial title for IndexedDataTable
comment:5 Changed 6 years ago by pfleck
- Owner changed from pfleck to abeham
- Status changed from accepted to reviewing
Maybe we can come up with a version that does not need the generic IDataTable, however, I don't think this should be a big deal because the interface previously did not existed anyway.
In the long run (HL 4.0, maybe), we could also consider merging DataTable, IndexedDataTable and ScatterPlot into a single DataTable. Per default, we could assume that the x-value is the index of the data point, and also allow the user to explicitly specify another DataRow as x-axis to offer a similar behavior such as the ScatterPlot and IndexedDataTable currently do.
comment:6 follow-up: ↓ 8 Changed 6 years ago by abeham
- Owner changed from abeham to pfleck
- Status changed from reviewing to assigned
"Start at Index 0" is meaningless for IndexedDataRows is there a way to disable it?
Otherwise, the changes look good to me.
comment:7 Changed 6 years ago by pfleck
- Status changed from assigned to accepted
r16520 merged trunk into branch
comment:8 in reply to: ↑ 6 Changed 6 years ago by pfleck
Replying to abeham:
"Start at Index 0" is meaningless for IndexedDataRows is there a way to disable it?
Otherwise, the changes look good to me.
I checked if there are any quick ways to disable the checkbox, but we can't access the (Indexed)DataTable from the VisualProperties to determine whether to disable the "Start at Index 0" or not.
After discussing it with abeham, we will leave it as it currently is, and treat the issue eventually when we unify the ScatterPlot and DataTable.
comment:9 Changed 6 years ago by pfleck
- Added backwards compatibility for IndexedDataTable.VisualProperties.Title
- Removed syncing of ItemName and VisualProperties.Title when (Indexed)DataTable is active.
comment:10 Changed 6 years ago by pfleck
- Owner changed from pfleck to abeham
- Status changed from accepted to reviewing
comment:11 Changed 6 years ago by abeham
- Owner changed from abeham to pfleck
- Status changed from reviewing to readytorelease
Change r16524 looks good to me. Thanks
comment:12 Changed 6 years ago by pfleck
r16527 merged branch to trunk
comment:13 Changed 6 years ago by pfleck
- Version changed from branch to trunk
comment:14 Changed 6 years ago by gkronber
comment:15 Changed 6 years ago by pfleck
- Resolution set to done
- Status changed from readytorelease to closed
r16929: removed branch
r16148 branched trunk