From 1bf17c9275d57e558d648ed96cf86a900192d3a4 Mon Sep 17 00:00:00 2001 From: Jared Scheib Date: Wed, 15 Feb 2017 01:37:50 -0800 Subject: [PATCH] Fix y-axis graph padding --- ui/src/shared/parsing/getRangeForDygraph.js | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/ui/src/shared/parsing/getRangeForDygraph.js b/ui/src/shared/parsing/getRangeForDygraph.js index 204cfda90..16e5a5ceb 100644 --- a/ui/src/shared/parsing/getRangeForDygraph.js +++ b/ui/src/shared/parsing/getRangeForDygraph.js @@ -5,25 +5,21 @@ export default function getRange(timeSeries, override, value = null, rangeValue return override; } - const subtractPadding = (val) => +val - val * PADDING_FACTOR; - const addPadding = (val) => +val + val * PADDING_FACTOR; + const subtractPadding = (val) => +val - Math.abs(val * PADDING_FACTOR); + const addPadding = (val) => +val + Math.abs(val * PADDING_FACTOR); - const pad = (val, side) => { + const pad = (val) => { if (val === null || val === '') { return null; } - if (val < 0) { - return side === "top" ? subtractPadding(val) : addPadding(val); - } - - return side === "top" ? addPadding(val) : subtractPadding(val); + return val < 0 ? subtractPadding(val) : addPadding(val); }; const points = [ ...timeSeries, [null, pad(value)], - [null, pad(rangeValue, "top")], + [null, pad(rangeValue)], ]; const range = points.reduce(([min, max] = [], series) => {