This is an automated email from the ASF dual-hosted git repository.

sushuang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-echarts.git


The following commit(s) were added to refs/heads/master by this push:
     new 62d637c  (fix) make module export clearly
62d637c is described below

commit 62d637c790324365e5f4a715ccd2a60952b479bb
Author: sushuang <[email protected]>
AuthorDate: Wed Sep 19 17:26:46 2018 +0800

    (fix) make module export clearly
---
 echarts.all.js                  | 222 +++++++++++++++++++++++++++++++++++++---
 src/component/dataZoom.js       |  20 +---
 src/component/dataZoomInside.js |   4 -
 src/component/dataZoomSelect.js |   3 +-
 4 files changed, 216 insertions(+), 33 deletions(-)

diff --git a/echarts.all.js b/echarts.all.js
index d597c45..5c19639 100644
--- a/echarts.all.js
+++ b/echarts.all.js
@@ -22,13 +22,36 @@ export * from './src/export';
 
 import './src/component/dataset';
 
-// Import all charts and components
+
+
+// ----------------------------------------------
+// All of the modules that are allowed to be
+// imported are listed below.
+//
+// Users MUST NOT import other modules that are
+// not included in this list.
+// ----------------------------------------------
+
+
+
+// ----------------
+// Charts (series)
+// ----------------
+
+
+
+// All of the series types, for example:
+// chart.setOption({
+//     series: [{
+//         type: 'line' // or 'bar', 'pie', ...
+//     }]
+// });
+
 import './src/chart/line';
 import './src/chart/bar';
 import './src/chart/pie';
 import './src/chart/scatter';
 import './src/chart/radar';
-
 import './src/chart/map';
 import './src/chart/tree';
 import './src/chart/treemap';
@@ -47,29 +70,204 @@ import './src/chart/themeRiver';
 import './src/chart/sunburst';
 import './src/chart/custom';
 
-import './src/component/graphic';
+
+
+// -------------------
+// Coordinate systems
+// -------------------
+
+
+
+// All of the axis modules have been included in the
+// coordinate system module below, do not need to
+// make extra import.
+
+// `cartesian` coordinate system. For some historical
+// reasons, it is named as grid, for example:
+// chart.setOption({
+//     grid: {...},
+//     xAxis: {...},
+//     yAxis: {...},
+//     series: [{...}]
+// });
 import './src/component/grid';
-import './src/component/legendScroll';
-import './src/component/tooltip';
-import './src/component/axisPointer';
+
+// `polar` coordinate system, for example:
+// chart.setOption({
+//     polar: {...},
+//     radiusAxis: {...},
+//     angleAxis: {...},
+//     series: [{
+//         coordinateSystem: 'polar'
+//     }]
+// });
 import './src/component/polar';
+
+// `geo` coordinate system, for example:
+// chart.setOption({
+//     geo: {...},
+//     series: [{
+//         coordinateSystem: 'geo'
+//     }]
+// });
 import './src/component/geo';
-import './src/component/parallel';
+
+// `singleAxis` coordinate system (notice, it is a coordinate system
+// with only one axis, work for chart like theme river), for example:
+// chart.setOption({
+//     singleAxis: {...}
+//     series: [{type: 'themeRiver', ...}]
+// });
 import './src/component/singleAxis';
-import './src/component/brush';
+
+// `parallel` coordinate system, only work for parallel series, for example:
+// chart.setOption({
+//     parallel: {...},
+//     parallelAxis: [{...}, ...],
+//     series: [{
+//         type: 'parallel'
+//     }]
+// });
+import './src/component/parallel';
+
+// `calendar` coordinate system. for example,
+// chart.setOptionp({
+//     calendar: {...},
+//     series: [{
+//         coordinateSystem: 'calendar'
+//     }]
+// );
 import './src/component/calendar';
 
+
+
+// ------------------
+// Other components
+// ------------------
+
+
+
+// `grapchic` component, for example:
+// chart.setOption({
+//     graphic: {...}
+// });
+import './src/component/graphic';
+
+// `toolbox` component, for example:
+// chart.setOption({
+//     toolbox: {...}
+// });
+import './src/component/toolbox';
+
+// `tooltip` component, for example:
+// chart.setOption({
+//     tooltip: {...}
+// });
+import './src/component/tooltip';
+
+// `axisPointer` component, for example:
+// chart.setOption({
+//     tooltip: {axisPointer: {...}, ...}
+// });
+// Or
+// chart.setOption({
+//     axisPointer: {...}
+// });
+import './src/component/axisPointer';
+
+// `brush` component, for example:
+// chart.setOption({
+//     brush: {...}
+// });
+// Or
+// chart.setOption({
+//     tooltip: {feature: {brush: {...}}
+// })
+import './src/component/brush';
+
+// `title` component, for example:
+// chart.setOption({
+//     title: {...}
+// });
 import './src/component/title';
 
-import './src/component/dataZoom';
-import './src/component/visualMap';
+// `timeline` component, for example:
+// chart.setOption({
+//     timeline: {...}
+// });
+import './src/component/timeline';
 
+// `markPoint` component, for example:
+// chart.setOption({
+//     series: [{markPoint: {...}}]
+// });
 import './src/component/markPoint';
+
+// `markLine` component, for example:
+// chart.setOption({
+//     series: [{markLine: {...}}]
+// });
 import './src/component/markLine';
+
+// `markArea` component, for example:
+// chart.setOption({
+//     series: [{markArea: {...}}]
+// });
 import './src/component/markArea';
 
-import './src/component/timeline';
-import './src/component/toolbox';
+// `legend` component scrollable, for example:
+// chart.setOption({
+//     legend: {type: 'scroll'}
+// });
+import './src/component/legendScroll';
+
+// `legend` component not scrollable. for example:
+// chart.setOption({
+//     legend: {...}
+// });
+import './src/component/legend';
+
+// `dataZoom` component including both `dataZoomInside` and `dataZoomSlider`.
+import './src/component/dataZoom';
+
+// `dataZoom` component providing drag, pinch, wheel behaviors
+// inside coodinate system, for example:
+// chart.setOption({
+//     dataZoom: {type: 'inside'}
+// });
+import './src/component/dataZoomInside';
+
+// `dataZoom` component providing a slider bar, for example:
+// chart.setOption({
+//     dataZoom: {type: 'slider'}
+// });
+import './src/component/dataZoomSlider';
+
+// `dataZoom` component including both `visualMapContinuous` and 
`visualMapPiecewise`.
+import './src/component/visualMap';
 
+// `visualMap` component providing continuous bar, for example:
+// chart.setOption({
+//     visualMap: {type: 'continuous'}
+// });
+import './src/component/visualMapContinuous';
+
+// `visualMap` component providing pieces bar, for example:
+// chart.setOption({
+//     visualMap: {type: 'piecewise'}
+// });
+import './src/component/visualMapPiecewise';
+
+
+
+// -----------------
+// Render engines
+// -----------------
+
+
+
+// Provide IE 6,7,8 compatibility.
 import 'zrender/src/vml/vml';
+
+// Render via SVG rather than canvas.
 import 'zrender/src/svg/svg';
diff --git a/src/component/dataZoom.js b/src/component/dataZoom.js
index bafb820..8be3b02 100644
--- a/src/component/dataZoom.js
+++ b/src/component/dataZoom.js
@@ -17,20 +17,8 @@
 * under the License.
 */
 
-/**
- * DataZoom component entry
- */
+import './dataZoomSlider';
+import './dataZoomInside';
 
-import './dataZoom/typeDefaulter';
-
-import './dataZoom/DataZoomModel';
-import './dataZoom/DataZoomView';
-
-import './dataZoom/SliderZoomModel';
-import './dataZoom/SliderZoomView';
-
-import './dataZoom/InsideZoomModel';
-import './dataZoom/InsideZoomView';
-
-import './dataZoom/dataZoomProcessor';
-import './dataZoom/dataZoomAction';
+// Do not include './dataZoomSelect',
+// since it only work for toolbox dataZoom.
diff --git a/src/component/dataZoomInside.js b/src/component/dataZoomInside.js
index 385625e..b291f85 100644
--- a/src/component/dataZoomInside.js
+++ b/src/component/dataZoomInside.js
@@ -17,10 +17,6 @@
 * under the License.
 */
 
-/**
- * DataZoom component entry
- */
-
 import './dataZoom/typeDefaulter';
 
 import './dataZoom/DataZoomModel';
diff --git a/src/component/dataZoomSelect.js b/src/component/dataZoomSelect.js
index d96bb9a..6cab5f0 100644
--- a/src/component/dataZoomSelect.js
+++ b/src/component/dataZoomSelect.js
@@ -18,7 +18,8 @@
 */
 
 /**
- * DataZoom component entry
+ * Only work for toolbox dataZoom. User
+ * MUST NOT import this module directly.
  */
 
 import './dataZoom/typeDefaulter';


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to