From 84390da2e46562d3d4602314b5a7d9811c98d42d Mon Sep 17 00:00:00 2001 From: Frederic Junod Date: Thu, 15 May 2014 08:59:02 +0200 Subject: [PATCH 1/2] Rename ol.color.ENABLE_NAMED_COLORS in tests --- test/spec/ol/color.test.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/spec/ol/color.test.js b/test/spec/ol/color.test.js index f14ea58bb7..3a61bacffe 100644 --- a/test/spec/ol/color.test.js +++ b/test/spec/ol/color.test.js @@ -13,7 +13,7 @@ describe('ol.color', function() { ol.color.fromStringInternal_.restore(); }); - if (ol.color.ENABLE_NAMED_COLORS) { + if (ol.ENABLE_NAMED_COLORS) { it('can parse named colors', function() { expect(ol.color.fromString('red')).to.eql([255, 0, 0, 1]); }); @@ -36,7 +36,7 @@ describe('ol.color', function() { [255, 255, 0, 0.1]); }); - if (ol.color.ENABLE_NAMED_COLORS) { + if (ol.ENABLE_NAMED_COLORS) { it('caches parsed values', function() { var count = ol.color.fromStringInternal_.callCount; ol.color.fromString('aquamarine'); From b42f4e1b0cb3eeaecd3cd72b655a3d47ac30a3c8 Mon Sep 17 00:00:00 2001 From: Frederic Junod Date: Thu, 15 May 2014 08:59:47 +0200 Subject: [PATCH 2/2] Change ol.ENABLE_NAMED_COLORS to be false by default --- src/ol/ol.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ol/ol.js b/src/ol/ol.js index 97f362903c..68ab072af0 100644 --- a/src/ol/ol.js +++ b/src/ol/ol.js @@ -86,9 +86,9 @@ ol.ENABLE_IMAGE = true; /** * @define {boolean} Enable named colors. Enabling named colors adds about 3KB * uncompressed / 1.5KB compressed to the final build size. Default is - * `true`. + * `false`. */ -ol.ENABLE_NAMED_COLORS = true; +ol.ENABLE_NAMED_COLORS = false; /**