diff options
| -rw-r--r-- | app/static/js/makeChart.js | 8 | ||||
| -rw-r--r-- | app/static/js/makeChartMulti.js | 10 |
2 files changed, 17 insertions, 1 deletions
diff --git a/app/static/js/makeChart.js b/app/static/js/makeChart.js index b6e3de2..9b074f1 100644 --- a/app/static/js/makeChart.js +++ b/app/static/js/makeChart.js @@ -59,6 +59,14 @@ function baseChart( divWidth, divHeight ) { + /* Note: + * It may be tempting to scale the width/height of the div to be + * proportional to the size of the window. However this may cause some + * users with wide screens to perceive changes in the time series + * differently than others because the horizontal axis is more + * stretched out. It is therefore better to keep the size of the graph + * the same for all users. + */ if (divWidth === null || typeof divWidth === 'undefined') divWidth = 1000; if (divHeight === null || typeof divHeight === 'undefined') diff --git a/app/static/js/makeChartMulti.js b/app/static/js/makeChartMulti.js index d0e4b31..ff0ceff 100644 --- a/app/static/js/makeChartMulti.js +++ b/app/static/js/makeChartMulti.js @@ -98,8 +98,16 @@ function baseChart(selector, data, clickFunction, annotations, annotationFunctio var lineObjects = {}; var pointSets = {} + /* Note: + * It may be tempting to scale the width/height of the div to be + * proportional to the size of the window. However this may cause some + * users with wide screens to perceive changes in the time series + * differently than others because the horizontal axis is more + * stretched out. It is therefore better to keep the size of the graph + * the same for all users. + */ var lineHeight = 150; - lineWidth = 1000; + var lineWidth = 1000; var chartPadding = 30; |