Opened 9 years ago

Closed 9 years ago

#1253 closed enhancement (done)

The RunCollectionTabularView should display some statistical information about the selected cells.

Reported by: mkommend Owned by: mkommend
Priority: low Milestone: HeuristicLab 3.3.2
Component: Optimization.Views Version: 3.3.2
Keywords: Cc:

Description


Change History (13)

comment:1 Changed 9 years ago by mkommend

  • Status changed from new to accepted

comment:2 Changed 9 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

Added StatusStrip in StringConvertibleMatrixView to diplay statistical information with r4709.

comment:3 Changed 9 years ago by mkommend

  • Owner changed from swagner to mkommend
  • Status changed from reviewing to assigned

The StatusStrip should be shown for all views and the min and max value of the selected cells should be added.

comment:4 Changed 9 years ago by mkommend

  • Status changed from assigned to accepted

comment:5 Changed 9 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

Improved StringConvertibleMatrixView to show statistics of selected cells per default with r4720.

comment:6 follow-up: Changed 9 years ago by swagner

  • Owner changed from swagner to mkommend
  • Status changed from reviewing to assigned

I would suggest the following additional modifications:

  • It would be nice, if the statistics values can be selected and copied (maybe use text boxes instead of labels to show the values? Or maybe it is possible to enable selection for labels?).
  • In some cases the background of the statistics values seems to have a different color than the background of the control in which the StringConvertibleMatrixView is embedded (for example in the coordinates matrix of a TSP). Maybe we should not use a StatusStrip?
  • I would recommend to align the statistics values on the left hand side instead of centering them.
  • I would recommend to use always the same size (i.e. width) for displaying a statistical value. Currently the width of the whole statistics text changes quite heavily when selecting cells which results in some kind of "flickering" experience.
  • If no numeric cells are selected, no space has to be kept for the statistics. In this case the StringConvertibleMatrixView should occupy the whole available space.

comment:7 Changed 9 years ago by mkommend

Improved statistics section of StringConvertibleMatrixView with r4779.

comment:8 in reply to: ↑ 6 Changed 9 years ago by mkommend

Changes in r4779 adressing the following problems:

Replying to swagner:

  • It would be nice, if the statistics values can be selected and copied (maybe use text boxes instead of labels to show the values? Or maybe it is possible to enable selection for labels?).

Used one TextBox to enable the selection and copying of values.

  • In some cases the background of the statistics values seems to have a different color than the background of the control in which the StringConvertibleMatrixView is embedded (for example in the coordinates matrix of a TSP). Maybe we should not use a StatusStrip?

The TextBox has always the window color background. Although in the RunCollectionTabularView the default background is grey => different color.

  • I would recommend to align the statistics values on the left hand side instead of centering them.
  • I would recommend to use always the same size (i.e. width) for displaying a statistical value. Currently the width of the whole statistics text changes quite heavily when selecting cells which results in some kind of "flickering" experience.

The flickering has been minimized by using string.Format to guarantee the same number of characters.

  • If no numeric cells are selected, no space has to be kept for the statistics. In this case the StringConvertibleMatrixView should occupy the whole available space.

This would also yield to a flickering and has not been implemented.

comment:9 Changed 9 years ago by mkommend

  • Status changed from assigned to accepted

comment:10 Changed 9 years ago by mkommend

  • Owner changed from mkommend to swagner
  • Status changed from accepted to reviewing

comment:11 Changed 9 years ago by swagner

  • Milestone changed from HeuristicLab x.x.x to HeuristicLab 3.3.2

comment:12 Changed 9 years ago by swagner

  • Owner changed from swagner to mkommend
  • Status changed from reviewing to readytorelease

Thanks for implementing my suggestions. Great.

comment:13 Changed 9 years ago by swagner

  • Resolution set to done
  • Status changed from readytorelease to closed
  • Version changed from 3.3.1 to 3.3.2
Note: See TracTickets for help on using tickets.