Configuration Context Menu for DataTableView and ScatterPlotView

[pfleck]

The DataTableView and ScatterPlotView both offer a config dialog option, which is accessible when shown within a ViewHost.

These configuration dialogs should also be accessible via the context menu of the chart (along the export options), to be able to open the config dialog without the ViewHost.

Furthermore, the DataTableView configuration dialog is lacking the option to hide a data series in the legend (in the same way the ScatterPlot offers this option).

Additionally, an optional regression-line is added for the ScatterPlotView.

[pfleck]

r14435 Added new ToolStripMenuItem to the chart for DataTableView and ScatterPlotView to open the configuration dialog.

[pfleck]

r14436 Pre-commited changes of VS auto-generated code for easier review later. (re-ordering of control initialization, ...)

[pfleck]

r14437 Added a "isVisibleInLegendCheckBox" for data rows in DataTables.

[pfleck]

r14438 Hide the chart-title if the title is empty (to avoid unnecessary empty space)

[pfleck]

r14439 Moved the charting logic out of DataTableView and ScatterPlotView into new DataTableControl and ScatterPlotControl. This way, the new controls can be reused without the name-textbox etc from the NamedItemView base class.

[pfleck]

r14493 Added option in the ScatterPlotControl to show a regression curve for a ScatterPlotDataRow (linear, poly, exp, log, pow).

[pfleck]

r14516 Fixed a charting issue that wrongly interprets scatterplot data if all x-values are zero.

[pfleck]

r14519 Fixed an issue with ObjectDisposedExceptions by using a AsynchronousContentView as base class instead of a UserControl.

[abeham]

Reviewed r14435: The tool strip menu item should read "Configure Chart...". The points indicate that an application modal dialog is going to be opened.

Reviewed r14436:14439: ok

Reviewed r14493: The scatter plot will display an error if there are negative y values and you're using Logarithmic regression. There is nothing displayed if Exponential regression is chosen. Also given the script below, the regression line is not drawn up to the last point, if e.g. Polynomial or Linear regression is chosen:

    var dt = new ScatterPlot("Table", "");
    var row = new ScatterPlotDataRow() { Name = "Row", VisualProperties = { PointSize = 8 }};
    row.Points.AddRange(new [] { new Point2D<double>(0, -3), new Point2D<double>(1, -2), new Point2D<double>(2, -1),
                                 new Point2D<double>(3, -0.5), new Point2D<double>(4, -0.25), new Point2D<double>(5, -0.125) });
    dt.Rows.Add(row);
    vars.dt = dt;

Reviewed r14516: ok

Reviewed r14519: Maybe you can add IConfigureableView to DataTableControl and ScatterPlotControl now that it is an AsynchronousContentView? It's also rather unique to have a content view that has neither a view nor a content attribute. Thus, I think it is better to derive DataTableView from DataTableControl instead of NamedItemView and reimplement the name/description handling.

[pfleck]

r14982 Reverted the separation of the DataTable/ScatterPlotView in -View and -Control. Instead, a ShowName property is added to control whether the name-textbox and the info-label from the NamedItemView are shown.

[pfleck]

r14987

• Fixed problems with negative values for some regression types.
• Fixed issue where the regression line was not drawn up to the last point.
• Tool strip now reads "Configure Chart...".

[pfleck]

r14992 Fixed a potential infinite loop that creates infinite datapoints.

[abeham]

Reviewed r14982:

• I think ShowName can throw an IllegalCrossThreadException if accessed from a Non-UI thread.
• The new property is called ShowName, but actually is implemented to show only the chart and hide everything else (also controls added by a potential derived view). I would thus suggest to rename it to "ShowOnlyChart" or "ShowChartOnly". Also I would use a private backing field for the property instead of accessing UI controls in the getter or setter. Introduce a method for performing the setter action and check for invoke required there.

Reviewed r14987 and r14992: okay

[pfleck]

r15042 Renamed ShowName to ShowChartOnly, added a backing field and introduced a method to perform the visibility changes of the controls.

[abeham]

ok

[pfleck]

In r14982, I accidentally included some changes of #2715 in the change (moved the whole file); thus, this ticket will be released after #2715.

[pfleck]

Due to a svn-move operation of DataTable/ScatterPlotControl -> DataTable/ScatterPlotView (in r14982), changes of multiple tickets (#2713, #2715, #2765) were included in a single revision. Instead of a manually extracting the changes from that files, all three tickets will be merged and released simultaneously.

[pfleck]

r15097: Merged to stable

• 14435-14439,14493,14516,14519,14982,14987,14992,15042 (from #2713)
• 14457-14458,14508,14582,14740,14984,15068,15095 (from #2715)
• 14860-14861 (from #2765)