From 61f89bab6e400d775425467bf83ea4824c7e1cff Mon Sep 17 00:00:00 2001 From: Tom Payne Date: Thu, 20 Jun 2013 05:29:35 +0200 Subject: [PATCH] Prevent buttons from setting out-of-range brightness/contrast/saturation values --- examples/brightness-contrast.js | 6 +++--- examples/hue-saturation.js | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/examples/brightness-contrast.js b/examples/brightness-contrast.js index e6dac0eac7..c5726f3cca 100644 --- a/examples/brightness-contrast.js +++ b/examples/brightness-contrast.js @@ -29,7 +29,7 @@ function setResetBrightnessButtonHTML() { setResetBrightnessButtonHTML(); increaseBrightness.addEventListener('click', function() { - layer.setBrightness(layer.getBrightness() + 0.125); + layer.setBrightness(Math.min(layer.getBrightness() + 0.125, 1)); setResetBrightnessButtonHTML(); }, false); resetBrightness.addEventListener('click', function() { @@ -37,7 +37,7 @@ resetBrightness.addEventListener('click', function() { setResetBrightnessButtonHTML(); }, false); decreaseBrightness.addEventListener('click', function() { - layer.setBrightness(layer.getBrightness() - 0.125); + layer.setBrightness(Math.max(layer.getBrightness() - 0.125, -1)); setResetBrightnessButtonHTML(); }, false); @@ -59,6 +59,6 @@ resetContrast.addEventListener('click', function() { setResetContrastButtonHTML(); }, false); decreaseContrast.addEventListener('click', function() { - layer.setContrast(layer.getContrast() - 0.125); + layer.setContrast(Math.max(layer.getContrast() - 0.125, 0)); setResetContrastButtonHTML(); }, false); diff --git a/examples/hue-saturation.js b/examples/hue-saturation.js index 53049a4336..a2aba69fa1 100644 --- a/examples/hue-saturation.js +++ b/examples/hue-saturation.js @@ -63,6 +63,6 @@ resetSaturation.addEventListener('click', function() { setResetSaturationButtonHTML(); }, false); decreaseSaturation.addEventListener('click', function() { - layer.setSaturation(layer.getSaturation() - 0.25); + layer.setSaturation(Math.max(layer.getSaturation() - 0.25, 0)); setResetSaturationButtonHTML(); }, false);