search

Letractively / choosel

Automatically exported from code.google.com/p/choosel
0 stars 0 forks source link

Refactor Charting widgets #117

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The charting widgets are currently hard to test and to maintain. This 
refactoring should achieve:

- Clearly separated Protovis API with possibility to get a mock instatiation 
for testing
- Provide reusable Protovis widget that can be used in flexible ways
- Move all view-specific code into the view content displays
- Refactor and display the view content display to some degree

Original issue reported on code.google.com by Lars.Grammel on 8 Dec 2010 at 5:28

GoogleCodeExporter commented 8 years ago 
Moved the Protovis wrapper code into [client].ui.widget.protovis package and 
refactored the callback functions to remove the dependency to 
[client].views.chart. The code for the specific visualizations (e.g. bar chart, 
pie chart) has been moved into the corresponding view content displays.

Original comment by Lars.Grammel on 8 Dec 2010 at 7:20

GoogleCodeExporter commented 8 years ago

Original comment by Lars.Grammel on 9 Dec 2010 at 7:26

GoogleCodeExporter commented 8 years ago 
The ChartViewContentDisplay hierarchy has been refactored. Some refactoring to 
scatter plot, bar chart and pie chart have been made, but refactoring those 
widgets is deferred to issues that change these widgets.

Original comment by Lars.Grammel on 9 Feb 2011 at 4:15